-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Resolve warning unused parameter 'tag' [-Werror=unused-parameter]
#474
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
Conversation
Memory usage change @ 21b48a2
Click for full report table
Click for full report CSV
|
This seems like a very low risk change. Does it need anything further, or when is it expected to be merged?
I fixed this again, because it has been a couple years since I fixed it last. 😄
When I was linking all the similar fixes, I found my own again.
jmparatte
commented
May 8, 2025
Hi,
Would you add permanently the compilation option [-Werror=unused-parameter] ? Or add a compilation option in [Preferences...]
I'm interested by a new option, but I would also that Arduino source code be corrected - AVR new.cpp - to show a good code.
And for my own code, to be noticed of unused parameter and take actions.
Best regards.
Let's get this merged! 😄
Clears a warning is generated by the Arduino CLI when specifying
--warnings all
.