add way to declare parameter as unused#72
add way to declare parameter as unused#72mehlis wants to merge 1 commit intoRIOT-OS:masterfrom mehlis:unused-parameter-macro
Conversation
Riot lags some way to declare, that a function parameter is unused. This commit adds a file called "unused.h", which defines some nice macros. http://stackoverflow.com/questions/3599160/unused-parameter-warnings-in-c-code
|
Sorry, but I don't quite see the advantage in using |
|
Let me elaborate further why I'd prefer But |
|
On 07/18/13 12:48, René Kijewski wrote:
|
|
ok, I see. You two prefer an other way :) @Kijewski make a pull request and I' am totally with you in merging your code! after having fixed this class of problems, there's just one single warning in riot (-Wall and -Wextra enabled, gcc and clang), so we are on a good way to have a warning free system! |
|
On 07/18/13 14:22, Christian Mehlis wrote:
+1 for committing (void) stuff. |
|
current "master"+"unused fixes": native and msba2 are warning-free. msb-430h has some warnings. |
|
I agree with @Kijewski the mysterious @RIOT-notifier ;-). So, I guess we can close this one. |
RIOT: update to 2020.07
4 participants
Riot lags some way to declare, that a function parameter
is unused. This commit adds a file called "unused.h",
which defines some nice macros.
http://stackoverflow.com/questions/3599160/unused-parameter-warnings-in-c-code
this commit fixes #62 (the discussion wasn't finally over in this issue)