This is the mail archive of the
newlib@sourceware.org
mailing list for the newlib project.
Re: [PATCH v2] Support %m in printf functions
On Jan 9 18:22, Yaakov (Cygwin/X) wrote:
> On Mon, 2012-01-09 at 12:51 +0100, Corinna Vinschen wrote:
> > On Jan 6 01:00, Yaakov (Cygwin/X) wrote:
> > > On Sun, 2012-01-01 at 11:59 -0600, Yaakov (Cygwin/X) wrote:
> > > > This patch adds support for the "%m" conversion specifier in the printf
> > > > and wprintf functions, a GNU extension:
> > > >
> > > > http://www.kernel.org/doc/man-pages/online/pages/man3/printf.3.html
> > >
> > > Thank you all for the comments. The attached revised patch and STC
> > > should take them into account.
> > >
> > > The remaining issue is that of gating, specifically exactly how to do
> > > so. For the moment I'm using __CYGWIN__ until some agreement can be
> > > reached.
> >
> > Jeff, I'd like to get your input on this.
> >
> > 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.
Thanks,
Corinna
--
Corinna Vinschen
Cygwin Project Co-Leader
Red Hat