[BusyBox] Re: ifconfig: Clean up. [PATCH]

Glenn L. McGrath bug1 at iinet.net.au
Sun Jul 24 21:02:52 MDT 2005


On Sun, 24 Jul 2005 20:50:39 -0500
Rob Landley <rob at landley.net> wrote:

> The problem is I really hate adding extra #ifdefs, especially for
> something  that small.  For 1.0.1 this doesn't really need to be
> addressed at all, and  for 1.1 what would be really nice would be a
> general reorganization where we  have libbb.h have one #ifdef that set
> "do_cleanup" to 0 or 1, and then we  used if(do_cleanup) in the body
> of the code and let the compiler's dead code  elimination yank it for
> us...
> 
> Opinions?

Is it just a question of aesthetics ?

People will forever agrue about which style looks nicest, its just what
people are used to i think.

Best we can do is try and adhere to common practices (if there is one)
and be consistent through all our code, at least we can might get used
to it that way.



Glenn


More information about the busybox mailing list