Re: [PATCH 1/2] Fix additional use of 'j' printf length modifier.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


On 07/07/2011 12:12 AM, Jonathan Nieder wrote:
Herbert Xu wrote:

%j is part of ISO C99 and POSIX
*checks*  Oh, so it is!  Thanks; it's nice to learn the standard
offers more than expected.

Brian: fwiw gnulib printf seems to provide %j, though of course it's a
bit more heavyweight.  And of course there are alternative C libraries
--- e.g., glibc runs on Solaris.

C99 also guarantees the presence of imaxdiv, but arith_yacc.c happily
#ifdefs around it if the build environment lacks it.  I submitted other
patches in the same spirit (working around non-compliant vsnprintf) and they
were accepted.  Would a patch that retains the old code verbatim but
#ifdefs in an alternate implementation on legacy systems be more acceptable?

-- Brian
--
To unsubscribe from this list: send the line "unsubscribe dash" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

Powered by Linux