Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:77238
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain zend.com designates 209.85.220.177 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <528B8D9E-58FF-47DD-9B91-013F8844EC70@ajf.me>
References: <B08F027B-EACA-42A1-BAFD-8B0FB3135016@ajf.me>
	<CAFMT4NpNqexoD+5MuVYPRCwqwgw5Ft3n2t4aXSF52_wAkuNDAw@mail.gmail.com>
	<528B8D9E-58FF-47DD-9B91-013F8844EC70@ajf.me>
Date: Tue, 16 Sep 2014 13:16:18 +0400
Message-ID: <CA+9eiLsdRRn0RxHrx=QYbJpEmp5H80hs+oB8+1MhJ0i24kDK=g@mail.gmail.com>
To: Andrea Faulds <ajf@ajf.me>
Cc: Levi Morrison <levim@php.net>, PHP Developers Mailing List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11c2a9f23ed48a05032b34e7
Subject: Re: [PHP-DEV] [VOTE][RFC] Integer Semantics
From: dmitry@zend.com (Dmitry Stogov)

--001a11c2a9f23ed48a05032b34e7
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hi Andrea,

Shifts by negative number may make sense. (N << -1) =3D> (N >> 1)
At least receiving "false" from shift is not very pleasant.

In the patch you use SIZEOF_LONG. It probably should be changed to
SIZEOF_ZEND_LONG.

Thanks. Dmitry.


On Mon, Sep 15, 2014 at 8:31 PM, Andrea Faulds <ajf@ajf.me> wrote:

>
> On 15 Sep 2014, at 17:24, Levi Morrison <levim@php.net> wrote:
>
> >> This RFC has been put to a vote. It starts today (2014-09-14) and ends
> in a week=E2=80=99s time (2014-09-21).
> >>
> >> https://wiki.php.net/rfc/integer_semantics#vote
> >
> > A few people have asked why I voted no; the only reason is that
> > division by zero will return `false` and emit a warning. Integers can
> > be converted to floating point numbers and dividing by zero in IEEE
> > 754 is defined as INF. I would rather leave behavior undefined by
> > voting no on this RFC than by defining it to a value which does not
> > make sense to me. The value false will get converted to zero in any
> > additional math, which is not the same as INF.
>
> The RFC does not touch division by zero. That was a suggestion for a
> future RFC.
>
> I=E2=80=99ve now removed that section to avoid confusion.
> --
> Andrea Faulds
> http://ajf.me/
>
>
>
>
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>

--001a11c2a9f23ed48a05032b34e7--