Re: [PHP-DEV] [PATCH] - fix for 64 bit issues with OnUpdateInt

[email protected] (Jani Taskinen)
Newsgroups php.dev
Message-ID <[email protected]>
On Fri, 28 Feb 2003, Andi Gutmans wrote:

>At 04:28 PM 2/28/2003 +0100, Sascha Schumann wrote:
>> > So I think the fix of adding OnUpdateLong() is the correct fix.
>>
>>     I was under the impression that OnUpdateInt was actually
>>     expecting a long.  I remember changing some int's to long's
>>     to address 64 bit issues.  Do I remember this incorrectly?
>
>No, you are correct but misunderstood me. We should introduce 
>OnUpdateLong() for ppl using longs and use OnUpdateInt() for ppl who want 
>to use ints.

    So for 4.3.2, we add the OnUpdateLong() and replace
    all the calls to OnUpdateInt() to use that instead
    and we leave the OnUpdateInt() behaviour same as it was.
    This shouldn't cause BC problems then..? 
    
    And in 5.0.0 we change OnUpdateInt() to use ints.
    
    --Jani
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.