On Thu, 18 Mar 2004 13:25:58 -0500 Michael Jennings <[EMAIL PROTECTED]> babbled:
> On Thursday, 18 March 2004, at 19:17:45 (+0100),
> Kim Woelders wrote:
>
> > I have deliberately been taking out these things because I think
> > they are silly :)
>
> You may think they're silly, but the rest of us do not. (The "rest of
> us" being mandrake, myself, raster I believe, and a few others.)
>
> See USE_VAR() in LibAST for a portable way to do this.
>
> Of course, the BEST solution to the problem is to not have unused
> parameters being passed at all.
There were good reasons these went in originally - it was a small spate of bugs
thanks to copy & paste and not using params. ie:
move(void *thing, int x, int y) {
x_move(thing, x, x);
}
now... notice the "silly" bug here? it gets caught or warned if we dont use a
param - this was present in a few places in the code and it was decided to start
using the warnings and explicitly saying you do NOT use a variable somewhere
harmless like after a return.
that was the original reasoning behind it and it caught a few potential bugs
after that... yes it looks strange and yes you are right - there may be
"unreachable code" warnings generated (though i havent seen those yet) but it
has helped in the past - thus the reason to use it.
--
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler) [EMAIL PROTECTED]
熊耳 - 車君 (数田) [EMAIL PROTECTED]
Tokyo, Japan (東京 日本)
-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
_______________________________________________
enlightenment-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
