CONFIG_* defines

[Rob Landley]

On Wednesday 07 September 2005 02:44, Bernhard Fischer wrote:
> >I've been slowly switching things over to the ENABLE_* macros, which
> > instead of being defined or undefined are always defined, but to 0 or 1. 
> > That way they can be used in normal if() statements.  Since it's a
> > constant, the
>
> Yes, since you started to do that, i'm getting:
> applets/applets.c:447: warning: empty body in an if-statement
> applets/applets.c:455: warning: empty body in an if-statement
> applets/busybox.c:74: warning: empty body in an if-statement
> See e.g. allnoconfig.

I checked in a some fixes to scripts/config/confdata.c:

http://www.busybox.net/cgi-bin/viewcvs.cgi/trunk/busybox/scripts/config/confdata.c?rev=11329&view=log

Unfortunately, yet another example of how our make dependencies are horked is 
that not only does scripts/config not get rebuilt, but a "make clean" doesn't 
delete scripts/config/{conf,mconf,mkconfigs}

Do a "make clean", then delete those three files by hand, and then rebuild.

> >compiler's "dead code elimination" optimization (a very old and very
> > simple optimization, one even fabrice bellard's "tiny C compiler" does)
> > can chop them out.
>
> yes, tcc also has dead code elimination. It doesn't support
> variable-sized arrays, currently. It should be taught to deal with them
> as they are useful generally and also used in busybox. I remember i got
> some headache from starting to support tcc -E some time ago, though.

I keep meaning to play with it more than I have, but it's not like I have a 
shortage of things to do... :)

Rob