-
Notifications
You must be signed in to change notification settings - Fork 69
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
Resolve all warnings #27
Labels
help wanted
Extra attention is needed
Comments
Fixed |
I just built your latest changes. 6 warnings still remain. To see them all, ensure in Arduino IDE you set File --> Preferences --> "Compiler warnings" to "All".
|
On medical (or any safety-critical) equipment, generally zero warnings are allowed with the compiler using its strictest settings. |
novakps
added a commit
to makehaven/AmboVent
that referenced
this issue
Apr 14, 2020
AmboVent-1690-108#27 Fixed all warnings, warnings caused by implicit convert between s…
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I just built the latest (from my PR #20), and there are dozens of warnings to fix.
The text was updated successfully, but these errors were encountered: