Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:1105
Return-Path: <edink@proventum.net>
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Delivered-To: mailing list internals@lists.php.net
Received: (qmail 37894 invoked from network); 29 Apr 2003 14:53:49 -0000
Received: from unknown (HELO smtpgate.proventum.net) (195.216.217.135)
  by pb1.pair.com with SMTP; 29 Apr 2003 14:53:49 -0000
Received: from padedin (edin.intra.proventum.net [10.0.0.54])
	by smtpgate.proventum.net (Postfix) with SMTP
	id 7D8DD52943; Tue, 29 Apr 2003 16:53:48 +0200 (CEST)
Message-ID: <010601c30e5f$243a7a40$3600000a@intra.proventum.net>
To: "Sascha Schumann" <sascha@schumann.cx>
Cc: <internals@lists.php.net>
Date: Tue, 29 Apr 2003 16:53:48 +0200
Organization: Proventum Solutions ApS
MIME-Version: 1.0
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
X-Priority: 3
X-MSMail-Priority: Normal
X-Mailer: Microsoft Outlook Express 6.00.2800.1158
X-Mimeole: Produced By Microsoft MimeOLE V6.00.2800.1165
Subject: *= operator broken on Mac OS X and Windows [crit bug #23395]
From: edink@proventum.net ("Edin Kadribasic")

The following piece of code

$i = 1;
$i *= 10;
echo $i;

prints -2123890688 on Mac OS X, and 0 on Windows.

The following commit is where breakage occured:

http://cvs.php.net/diff.php/Zend/zend_operators.c?r1=1.129.2.1&r2=1.129.2.2&ty=u

I suggest for this patch to be reverted since the commit message
reads "Speed up, simplify, use ZEND_SIGNED_MULTIPLY_LONG in
mul_operation" which doesn't sound like a bug fix that should be in
the stable branch.

Edin