Ilia? Greg?
Any comments?
On 14.08.2007 15:42, Antony Dovgal wrote:
This patch breaks PEAR phar install on 64bit.
From what I can see in the source code, it compares data in the phar with
the result of sprintf("%u"..), which is different on 32bit and 64bit because of
this patch.
Either phar should be changed to detect this situation or this patch should be
reverted.
-------- Original Message --------
Subject: [PHP-CVS] cvs: php-src /ext/standard formatted_print.c
Date: Sun, 03 Jun 2007 09:11:52 -0000
From: Brian Shire <[EMAIL PROTECTED]>
To: [EMAIL PROTECTED]
shire Sun Jun 3 09:11:52 2007 UTC
Modified files:
/php-src/ext/standard formatted_print.c
Log:
Change printf %u behavior so that it does not truncate numbers at 32-bits.
(Reported by Aditya Agarwal.)
http://cvs.php.net/viewvc.cgi/php-src/ext/standard/formatted_print.c?r1=1.100&r2=1.101&diff_format=u
Index: php-src/ext/standard/formatted_print.c
diff -u php-src/ext/standard/formatted_print.c:1.100
php-src/ext/standard/formatted_print.c:1.101
--- php-src/ext/standard/formatted_print.c:1.100 Thu May 17 17:29:09 2007
+++ php-src/ext/standard/formatted_print.c Sun Jun 3 09:11:52 2007
@@ -16,7 +16,7 @@
+----------------------------------------------------------------------+
*/
-/* $Id: formatted_print.c,v 1.100 2007/05/17 17:29:09 tony2001 Exp $ */
+/* $Id: formatted_print.c,v 1.101 2007/06/03 09:11:52 shire Exp $ */
#include <math.h> /* modf() */
#include "php.h"
@@ -278,7 +278,7 @@
PRINTF_DEBUG(("sprintf: appenduint(%x, %x, %x, %d, %d, '%c', %d)\n",
*buffer, pos, size, number, width, padding,
alignment));
- magn = (unsigned int) number;
+ magn = (unsigned long) number;
/* Can't right-pad 0's on integers */
if (alignment == 0 && padding == '0') padding = ' ';
--
Wbr,
Antony Dovgal
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
