Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Compiler warnings #15090

Closed
LineF opened this issue Aug 29, 2019 · 13 comments
Closed

[BUG] Compiler warnings #15090

LineF opened this issue Aug 29, 2019 · 13 comments

Comments

@LineF
Copy link
Contributor

LineF commented Aug 29, 2019

In bugfix-2.0.x branch there is a compiler warning issue since a longer time (i think somewhere in 2018):

In many compiled programs I get following warning (Arduino IDE 1.8.7):

===============
In file included from C:\Users\Martin\AppData\Local\Temp\arduino_build_976427\sketch\src\gcode\temperature\M140_M190.cpp:30:0:

c:\users\martin\appdata\local\temp\arduino_build_976427\sketch\src\lcd\ultralcd.h:343:29: warning: extra qualification 'MarlinUI::' on member 'expire_status_ms' [-fpermissive]

         static millis_t MarlinUI::expire_status_ms; // = 0

===============

Maybe someone can find the issue...

Thanks,
Martin

@boelle
Copy link
Contributor

boelle commented Aug 29, 2019

or 1.9 or vs code with platform.io extension

@LineF
Copy link
Contributor Author

LineF commented Aug 29, 2019

no, IDE 1.9.0 beta doesn't help - the warning is still present.
What do you mean with "vs code with platform.io extension" ?

@boelle
Copy link
Contributor

boelle commented Aug 29, 2019

@LineF
Copy link
Contributor Author

LineF commented Aug 29, 2019

@Ludy87: the fix solved the problem!
Thx a lot!

@Ludy87
Copy link
Contributor

Ludy87 commented Aug 29, 2019

@boelle i can confirm

@boelle
Copy link
Contributor

boelle commented Aug 29, 2019

@Ludy87 oki, we can close this one when the PR is merged

now you just need to make fix's for the other bugs :-D

@Ludy87
Copy link
Contributor

Ludy87 commented Aug 29, 2019

@boelle Which mistakes are that?

@boelle
Copy link
Contributor

boelle commented Aug 29, 2019

#15086
#15084
#15083

just examples.... :-D

@Ludy87
Copy link
Contributor

Ludy87 commented Aug 29, 2019

For #15086 I already wrote a PR ;-P

@Ludy87
Copy link
Contributor

Ludy87 commented Aug 29, 2019

@boelle
then it can be done here, Scott has brought the PR together

@boelle
Copy link
Contributor

boelle commented Aug 30, 2019

@Ludy87 it was meant like a joke :-D we have a lut of bugs that needs a smart head or maybe 10 smart heads

but of course an issue list without bugs is never going to happen, when new things are added the risk of bugs is there

@boelle
Copy link
Contributor

boelle commented Aug 30, 2019

pr is merged, closing

@boelle boelle closed this as completed Aug 30, 2019
@github-actions
Copy link

github-actions bot commented Jul 4, 2020

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Jul 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants