-
-
Notifications
You must be signed in to change notification settings - Fork 7k
Clean up examples to remove warnings [imported] #693
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
Comments
Done:
WontFix:
String txtMsg = ""; // a string for incoming text
-int lastStringLength = txtMsg.length(); // previous length of the String
+size_t lastStringLength = txtMsg.length(); // previous length of the String and the return type of To Do
|
cmaglie
pushed a commit
that referenced
this issue
Jul 6, 2017
- Avoids confusing beginners who assume there must be some reason for this useless line of code. - Fixes "warning: unused variable 'currentTime'" compiler warning. Closes #693
s0ren
pushed a commit
to STU-IT/arduino17-18
that referenced
this issue
Dec 6, 2017
- Avoids confusing beginners who assume there must be some reason for this useless line of code. - Fixes "warning: unused variable 'currentTime'" compiler warning. Closes arduino/Arduino#693
s0ren
pushed a commit
to STU-IT/arduino17-18
that referenced
this issue
Dec 6, 2017
…ample Fixes: warning: comparison between signed and unsigned integer expressions compiler warning. Closes arduino/Arduino#693 (in conjunction with arduino/Arduino#6448)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This is Issue 693 moved from a Google Code project.
Added by 2011-10-24T01:49:13.000Z by maniac...@ymail.com.
Please review that bug for more context and additional comments, but update this bug.
Original labels: Type-Defect, Priority-Medium, Component-Examples
Original description
Some examples have warnings which indicate where a few clean-ups could be made. This patch fixes all those.
(If it doesn't apply cleanly let me know, I'll regenerate one from a clean repo)
The text was updated successfully, but these errors were encountered: