Init Patch for review and evaluation

Eugene Bordenkircher eugebo at gmail.com
Fri Dec 28 02:04:12 UTC 2007


On Thu, 2007-12-27 at 13:20 -0500, Paul Fox wrote:
> > > #endif
>  > > in the future, please try to utilize the first form
>  > > -mike
>  > 
>  > Thanks for the heads up.  I'll make sure to do this in the future.   I'm
>  > assuming I probably should have used CONFIG_... instead of ENABLE...
>  > then.  I intended it to be a compile time feature to avoid bloat since
>  > most people will not need the feature.
> 
> no -- the point is that
>     if (ENABLE_XXX)
> is the same as 
>     if (0)
> or
>     if (1)
> 
> and the optimizer will completely eliminate the code in the first
> case, so the effect is the same as "#if CONFIG_XXX".  but since
> the code is "visible" to the compiler, compilation errors
> introduced by changes elsewhere (usually in header files) will be
> caught more quickly.
> 
> paul
> =---------------------
>  paul fox, pgf at brightstareng.com
> _______________________________________________
> busybox mailing list
> busybox at busybox.net
> http://busybox.net/cgi-bin/mailman/listinfo/busybox

Neat idea.  The thought never occurred to me.  I like it though. 

Thanks for the explanation.




More information about the busybox mailing list