Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:43089
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain googlemail.com designates 64.233.182.188 as permitted sender)
DomainKey-Status: bad
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=googlemail.com; s=gamma;
        h=message-id:date:from:user-agent:mime-version:to:cc:subject
         :references:in-reply-to:content-type:content-transfer-encoding;
        b=ZQouSoFQc+2MUGTuXgR+gHzgbQlq1UNS35oNq3jIDz5ujuDtqtepm13F8TzsSqAcy+
         swdb8SJgad83E/PSo2gLv0/lvbXesrOg8AKhqK8xtPSBlTjBhlWhUwTUNgvuHbtde1Q4
         sCYidR2SbOR3IyRMykrpjTwMi7TiDjWSPTfl0=
Message-ID: <499BF0CF.3000108@googlemail.com>
Date: Wed, 18 Feb 2009 11:28:15 +0000
User-Agent: Thunderbird 2.0.0.19 (X11/20090105)
MIME-Version: 1.0
To: Moriyoshi Koizumi <mozo@mozo.jp>
CC: Ilia Alshanetsky <ilia@prohost.org>, 
 internals Mailing List <internals@lists.php.net>,
 Dmitry Stogov <dmitry@zend.com>
References: <cd1fb7540902120848l43abcc7i34d24426ca456107@mail.gmail.com>	 <cd1fb7540902121256vb5c27caoe3246abde1aa62fb@mail.gmail.com>	 <6D771262-F1BD-43B7-93FC-BF9D47D15F49@prohost.org> <cd1fb7540902140027q4506c64ft28c1f1be4e49ccbe@mail.gmail.com>
In-Reply-To: <cd1fb7540902140027q4506c64ft28c1f1be4e49ccbe@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Re: Bug #46701
From: zoe.slattery@googlemail.com (zoe)

Moriyoshi Koizumi wrote:
> Please don't even think of backporting. This will definitely break a
> lot of things, and this kind of thing must not be done in a minor
> release.
>
>   
--snip--

>>>> I guess the patch relies on the 5.3's DVAL_TO_LVAL behavior that was
>>>> changed by the fix for bug #42868, right?
>>>> If so, this patch shouldn't be MFH'ed as the #42868 patch was not
>>>> merged although I didn't remember any discussion on this.
>>>>
>>>> See also: http://marc.info/?l=php-internals&m=120799720922202&w=2
>>>>
>>>>         
Hey all

I'm sorry - I should have replied to this before since I was responsible 
for raising #42868.  I didn't do a good job at explaining what the issue 
was in that bug, mainly because I didn't know what it was when I 
started. The central problem is that PHP's behaviour on casting double 
to int defaults to whatever the underlying C environment does. On 
Windows and Linux (all of the versions that I looked at) this turns out 
to be a simple truncation of the last 32 bits. Unfortunately the C 
behaviour is 'undefined' (Kernigan and Ritchie, page 197, A6.3). The 
issue that I found in #42868 was that on the Mac the casting behaviour 
is completely different so many of the PHP tests failed. I believe that 
PHP should behave in a platform independent way - that is what the fix 
to #42868 achieves. It is also fair to say that any applications that 
depend on the overflow behaviour in PHP 5.2 cannot be guaranteed to run 
on any platform.

Zoe
>>>> Regards,
>>>> Moriyoshi
>>>>
>>>>         
>>> --
>>> PHP Internals - PHP Runtime Development Mailing List
>>> To unsubscribe, visit: http://www.php.net/unsub.php
>>>
>>>       
>> Ilia Alshanetsky
>>
>>
>>
>>
>>
>>     
>
>