Re: cvs: php-src /main snprintf.c

On 04/13/2007 02:47 AM, Marcus Boerger wrote:

Hello Antony, under windows, where this is coming from (iirc), there is "__int64". Is that mapped to SIZEOF_LONG_LONG and don't you need to check for the actual value being 8?

Ok, so it's always defined on Windows. But there are also other platforms.

Friday, April 13, 2007, 12:00:56 AM, you wrote:

tony2001                Thu Apr 12 22:00:56 2007 UTC

  Modified files:                  /php-src/main       snprintf.c   Log:

add support for %I64(u|d) to sprintf() utilities http://cvs.php.net/viewvc.cgi/php-src/main/snprintf.c?r1=1.54&r2=1.55&diff_format=u Index: php-src/main/snprintf.c diff -u php-src/main/snprintf.c:1.54 php-src/main/snprintf.c:1.55

--- php-src/main/snprintf.c:1.54        Mon Apr  9 15:35:49 2007
+++ php-src/main/snprintf.c     Thu Apr 12 22:00:56 2007

@@ -16,7 +16,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: snprintf.c,v 1.54 2007/04/09 15:35:49 dmitry Exp $ */ +/* $Id: snprintf.c,v 1.55 2007/04/12 22:00:56 tony2001 Exp $ */ #include "php.h" @@ -706,6 +706,16 @@

                                        fmt++;
                                        modifier = LM_LONG_DOUBLE;
                                        break;
+                               case 'I':
+                                       fmt++;

+#if SIZEOF_LONG_LONG

+                                       if (*fmt == '6' && *(fmt+1) == '4') {
+                                               fmt += 2;
+                                               modifier = LM_LONG_LONG;
+                                       } else

+#endif

+                                               modifier = LM_LONG;
+                                       break;
                                case 'l':
                                        fmt++;

#if SIZEOF_LONG_LONG

Best regards, Marcus

-- Wbr, Antony Dovgal