Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:67601
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain hristov.com from 91.196.124.214 cause and error)
Message-ID: <51A9C4A6.3000802@hristov.com>
Date: Sat, 01 Jun 2013 11:53:42 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6
MIME-Version: 1.0
To: Anthony Ferrara <ircmaxell@gmail.com>
CC: Derick Rethans <derick@php.net>, 
 "internals@lists.php.net" <internals@lists.php.net>
References: <CAAyV7nG126YkpOF-EShHbf=4rgX7gQ2VE0ShRh0-50_Yr--h4g@mail.gmail.com> <6dae6c81-3e50-42f3-9a7e-87eedf8861d4@email.android.com> <CAAyV7nEOvvuLnrz0bO=gvb6zJWms+X9EcbR_77Op00W5s=MHeA@mail.gmail.com>
In-Reply-To: <CAAyV7nEOvvuLnrz0bO=gvb6zJWms+X9EcbR_77Op00W5s=MHeA@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] 5.NEXT Integer and String type modifications
From: php@hristov.com (Andrey Hristov)

  Hi,
On 05/31/2013 10:03 PM, Anthony Ferrara wrote:
> Derick,
>
>
> In principle I think this is great thing to do. Not having a 64 bit type is
>> annoying. I'm a bit curious on how this is going to work with all sorts of
>> object wrappers that are now in place as workaround. And casting int64 to
>> int32 needs to very well looked at as well.
>>
>>
> As far as the casting, my first reaction would be to raise an
> E_ENGINE_NOTICE on data loss (casting from int64 to int32 with ints > 32
> bit) and then adjusting the value to the nearest representable value
> (LONG_MAX or LONG_MIN). In other words, it may need to be more than a
> simple cast (an inline function perhaps)...
>
> As far as object wrappers, any particular examples that you're thinking of?
>
> Thanks for the thoughts
>
> Anthony
>
what about new type IS_LONG64, new field in union and new set of macros 
for this type. New extensions or rewritten extensions will use the new 
macros. In 2-3 major versions, 5.8 for example, old macros will be 
dropped. Enough time extensions to be ported to the new macros.

Best,
Andrey