[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