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]