[PATCH] stdio: Add %m

Rob Landley rob at landley.net
Tue Jun 6 23:22:58 UTC 2006


On Monday 05 June 2006 12:49 pm, Shaun Jackman wrote:
> In particular, busybox uses %m and the maintainers will not accept
> patches converting this format to %s and strerror(errno) due to the
> latter taking more space by making, typically, two additional function
> calls -- one to __errno_location and one to strerror.

Plus the extra code to load another argument onto the stack.  It adds up.

Also because although the result looks bad, it isn't a build break or a loss 
of functionality in the non-error case.  (Worst case scenario you just don't 
see the error details.)

> Cheers,
> Shaun

Rob
-- 
Never bet against the cheap plastic solution.



More information about the busybox mailing list