This is the mail archive of the
newlib@sourceware.org
mailing list for the newlib project.
Re: [PATCH] stdio: Add %m
- From: Rob Landley <rob at landley dot net>
- To: busybox at busybox dot net, Shaun Jackman <sjackman at gmail dot com>
- Cc: newlib at sources dot redhat dot com
- Date: Tue, 6 Jun 2006 19:22:58 -0400
- Subject: Re: [PATCH] stdio: Add %m
- References: <7f45d9390606050949u84cbcd6yd01823e96f8a621@mail.gmail.com>
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.