[PATCH v2] Support %m in printf functions

Yaakov (Cygwin/X) yselkowitz@users.sourceforge.net
Thu Jan 12 11:35:00 GMT 2012


On Thu, 2012-01-12 at 11:09 +0100, Corinna Vinschen wrote:
> On Jan  9 18:22, Yaakov (Cygwin/X) wrote:
> > On Mon, 2012-01-09 at 12:51 +0100, Corinna Vinschen wrote:
> > > IMHO, it doesn't feel right to add __CYGWIN__ here since the %m
> > > functionality is sufficiently target agnostic to be useful for everyone
> > > who would like to use it.  I guess _GLIBC_EXTENSION sounds about right,
> > > but maybe it should be _NO_GLIBC_EXTENSIONS instead and only set it
> > > for builds which explicitely don't want them?
> > 
> > What about #if _ELIX_LEVEL >= 4?
> 
> I don't think that's feasible.  Let's use the _GLIBC_EXTENSION macro.
> For Cygwin we should set it in winsup/cygwin/include/cygwin/config.h.

Revised patch for newlib and patch for winsup/cygwin attached.


Yaakov
Cygwin/X

-------------- next part --------------
A non-text attachment was scrubbed...
Name: newlib-printf-m.patch
Type: text/x-patch
Size: 3759 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20120112/916d4da6/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cygwin-glibc-exts.patch
Type: text/x-patch
Size: 747 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20120112/916d4da6/attachment-0001.bin>


More information about the Newlib mailing list