Init Patch for review and evaluation

Paul Fox pgf at brightstareng.com
Thu Dec 27 10:20:27 PST 2007


 > > #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


More information about the busybox mailing list