[patch] size optimization for setarch.c, fix personality applet-alias

Rob Landley rob at landley.net
Mon Mar 20 02:15:04 UTC 2006


On Saturday 18 March 2006 1:34 pm, Bernhard Fischer wrote:
> Hi,
>
>    text	   data	    bss	    dec	    hex	filename
>     151	      0	      0	    151	     97	setarch.o.orig
>     111	      0	      0	    111	     6f	setarch.o
>
>
> The aliases for "linux64" and "linux32" were missing from the applet
> list (see hunk against Config.in) which is printed when executing plain
> ./busybox. This snippet maybe something we want also for the stable
> branch as it could be considered a documentation fix.

Is anything, anywhere, actually warning about argc being unused?  (Warning 
about locally declared variables being unused, sure.  But function arguments 
are often externally imposed.  Warning about them being unused is a -pedantic 
at best.)

> Rob, if you agree then please make sure the part which touches
> Config.in makes it into svn before the release. TIA.
>
>
> I intend to check this in for the new trunk after tuesday unless
> somebody objects..

Silence is neither assent nor objection, although shrinking 0x40 bytes off an 
applet is generally a good sign. :)

I didn't check email for most of the weekend.  Went and saw a movie, bought 
some CD-Rewriteables to try to write my own cd burning software with, watched 
Fade play The Sims, petted cats, read a disappointing book, then found my 
email server was down and Eric was at Lunacon...

(I don't update landley.livejournal.com much, probably should....)

Rob
-- 
Never bet against the cheap plastic solution.



More information about the busybox mailing list