Re: cvs: php-src(PHP_5_2) /ext/mbstring/oniguruma regposerr.c

From: Date: Tue, 06 Mar 2007 01:00:04 +0000
Subject: Re: cvs: php-src(PHP_5_2) /ext/mbstring/oniguruma regposerr.c
References: 1  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-43420@lists.php.net to get a copy of this message
This is not going to work - sprintf has format as second parameter. Also, I thought PHP does define snprintf for Windows? Marcus Boerger wrote:
helly Tue Mar 6 00:56:42 2007 UTC
  Modified files:              (Branch: PHP_5_2)
    /php-src/ext/mbstring/oniguruma	regposerr.c   Log:
- MFH Revert back to sprintf (we could define snprintf for windows but we don't) http://cvs.php.net/viewvc.cgi/php-src/ext/mbstring/oniguruma/regposerr.c?r1=1.3.4.1&r2=1.3.4.2&diff_format=u Index: php-src/ext/mbstring/oniguruma/regposerr.c diff -u php-src/ext/mbstring/oniguruma/regposerr.c:1.3.4.1 php-src/ext/mbstring/oniguruma/regposerr.c:1.3.4.2 --- php-src/ext/mbstring/oniguruma/regposerr.c:1.3.4.1 Sat Feb 24 02:17:24 2007 +++ php-src/ext/mbstring/oniguruma/regposerr.c Tue Mar 6 00:56:42 2007 @@ -76,7 +76,7 @@
     s = "";
} else {
-    snprintf(tbuf, sizeof(tbuf), "undefined error code (%d)", posix_ecode);
+    sprintf(tbuf, sizeof(tbuf), "undefined error code (%d)", posix_ecode);
     s = tbuf;
}
-- Stanislav Malyshev, Zend Products Engineer stas@zend.com http://www.zend.com/

« previous php.cvs (#43420) next »