[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [patch #5822] Handle unused arguments
From: |
Simon Goldschmidt |
Subject: |
[lwip-devel] [patch #5822] Handle unused arguments |
Date: |
Mon, 16 Apr 2007 19:47:40 +0000 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.8.1.3) Gecko/20070309 Firefox/2.0.0.3 |
Update of patch #5822 (project lwip):
Status: In Progress => Done
Open/Closed: Open => Closed
_______________________________________________________
Follow-up Comment #12:
re comment #11:
Furtunately, gcc does not warn if you cast the unused variable to (void).
Either most of the users use gcc or many other compilers do it like this (or
don't warn at all?). But casting to void (as it is now) was in the code
already. This 'patch' only moves the casting into a define, which gives us a)
cleaner code and b) the possibility to adopt to other compilers.
Unfortunately, type information is not included...
The downside to your approach is that the statement (if not optimized away)
generates code, while '(void)arg;' doesn't (even with '-O0' for gcc).
If you want to change it, though, the places to change are now easy to find
;-)
Closing this as done, as I didn't find more appearences.
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/patch/?5822>
_______________________________________________
Nachricht geschickt von/durch Savannah
http://savannah.nongnu.org/