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