Use our own snprintf() only if NLS is enabled, and support %qd and %I64d.

diff --git a/configure b/configure
index 6414cb41864087166c2e8fe088105f4b8a77cc2a..6fc23b53442ea9b46db187719c4818f5d115767a 100755 (executable)
--- a/configure
+++ b/configure
@@ -14527,7 +14527,8 @@ fi
 
 
 # Force use of our snprintf if system's doesn't do arg control
-if test $pgac_need_repl_snprintf = no; then
+# This feature is used by NLS
+if test "$enable_nls" = yes -a $pgac_need_repl_snprintf = no; then
   echo "$as_me:$LINENO: checking whether printf supports argument control" >&5
 echo $ECHO_N "checking whether printf supports argument control... $ECHO_C" >&6
 if test "${pgac_cv_printf_arg_control+set}" = set; then

diff --git a/configure.in b/configure.in
index 073ea042f632019e2ff92af9e44482cb5a0b2301..43dce5d3e21aec1d85e680bdca3773e34cf8f975 100644 (file)
--- a/configure.in
+++ b/configure.in
@@ -1,5 +1,5 @@
 dnl Process this file with autoconf to produce a configure script.
-dnl $PostgreSQL: pgsql/configure.in,v 1.403 2005/02/28 20:55:18 tgl Exp $
+dnl $PostgreSQL: pgsql/configure.in,v 1.404 2005/03/02 14:48:22 momjian Exp $
 dnl
 dnl Developers, please strive to achieve this order:
 dnl
@@ -1067,7 +1067,8 @@ AC_MSG_ERROR([[
 
 
 # Force use of our snprintf if system's doesn't do arg control
-if test $pgac_need_repl_snprintf = no; then
+# This feature is used by NLS
+if test "$enable_nls" = yes -a $pgac_need_repl_snprintf = no; then
   PGAC_FUNC_PRINTF_ARG_CONTROL
   if test $pgac_cv_printf_arg_control != yes ; then
     pgac_need_repl_snprintf=yes

diff --git a/src/port/snprintf.c b/src/port/snprintf.c
index cba8ad5b1cc73efeba0dfcddb6ac601f75f2c445..64486393646a0dfea258d35e77acbc245f9840b3 100644 (file)
--- a/src/port/snprintf.c
+++ b/src/port/snprintf.c
@@ -65,7 +65,7 @@
  * causing nasty effects.
  **************************************************************/
 
-/*static char _id[] = "$PostgreSQL: pgsql/src/port/snprintf.c,v 1.12 2005/03/02 05:22:22 momjian Exp $";*/
+/*static char _id[] = "$PostgreSQL: pgsql/src/port/snprintf.c,v 1.13 2005/03/02 14:48:22 momjian Exp $";*/
 
 int            snprintf(char *str, size_t count, const char *fmt,...);
 int            vsnprintf(char *str, size_t count, const char *fmt, va_list args);
@@ -259,6 +259,23 @@ dopr(char *buffer, const char *format, va_list args, char *end)
                        else
                            longflag = 1;
                        goto nextch;
+                   /*
+                    *  We might export this to client apps so we should
+                    *  support 'qd' and 'I64d'(MinGW) also in case the
+                    *  native version does.
+                    */
+                   case 'q':
+                       longlongflag = 1;
+                       longflag = 1;
+                       goto nextch;
+                   case 'I':
+                       if (*(format+1) == '6' && *(format+2) == '4')
+                       {
+                           format += 2;
+                           longlongflag = 1;
+                           longflag = 1;
+                           goto nextch;
+                       }                   
                    case 'u':
                    case 'U':
                        /* fmtnum(value,base,dosign,ljust,len,zpad,&output) */