[PATCH] hdparm new option parsing

Rob Landley rob at landley.net
Sun May 7 21:45:01 PDT 2006


On Sunday 07 May 2006 3:28 pm, Tito wrote:
> New version of the previous patch with the ENABLE_ and USE_ stuff.
> Enjoy.

Well, it's still a little ugly but it's better than what was there before (and 
make bloatcheck says it saves 1440 bytes), so I've applied it.

Having a test as the first parameter of a function you call unconditionally is 
kinda ugly.  Is having an extra parameter in the function actually cheaper 
than an if statement?  The best solution I can come up with off the top of my 
head is table driven and I haven't got time to go off on that tangent right 
now, so...

Rob
-- 
Never bet against the cheap plastic solution.


More information about the busybox mailing list