-
Notifications
You must be signed in to change notification settings - Fork 107
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
Fix unused function and missing prototype warnings #628
Conversation
//DEBUG displayWaypoints(); | ||
DEBUG displayWaypoints(); | ||
|
||
#ifdef LOG_LIGHTS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The indentation looks a little off here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed the indentation on the logLights()
line. Having no indentation for the preprocessor directives seems standard though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I presume @tmewett means he prefers the new indentation, rather than wanting no indentation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mean that I agree it's typical for preprocessor directives to have no indentation - diff as-is looks good to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just the indentation
Some of these warnings were directly introduced in tmewett#593; others are due to other PRs that were merged between when I made that PR and when it was merged. Some of these are fixed with new `#define`s; this also makes `displayLoops` and `displayChokeMap` static and always runs them in wizard mode when '~' is pressed, along with `displayMachines` and `displayWaypoints`. This also improves those functions a bit, so it's easier to tell which output is from which function.
71eca4f
to
5e48c19
Compare
Some of these warnings were directly introduced in #593; others are due to other PRs that were merged between when I made that PR and when it was merged.
Some of these are fixed with new
#define
s; this also makesdisplayLoops
anddisplayChokeMap
static and always runs them in wizard mode when '~' is pressed, along withdisplayMachines
anddisplayWaypoints
. This also improves those functions a bit, so it's easier to tell which output is from which function.