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

> A single API is probably sufficient but I can understand that extension > writers might want to use ints. The INI flag might have nothing to do with > an internal zval and an int might be more than enough. > I don't really mind but I wouldn't introduce OnUpdateInteger. > I guess we should eith

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

In message <[EMAIL PROTECTED]> on Wed, Mar 05, 2003 at 07:35:49AM +0200, Andi Gutmans wrote: > At 01:29 PM 3/5/2003 +0800, James Devenish wrote: > >In message <[EMAIL PROTECTED]> > >on Wed, Mar 05, 2003 at 07:24:20AM +0200, Andi Gutmans wrote: > >Hasn't worked so far ;) > > Well we're talking abou

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

At 01:29 PM 3/5/2003 +0800, James Devenish wrote: In message <[EMAIL PROTECTED]> on Wed, Mar 05, 2003 at 07:24:20AM +0200, Andi Gutmans wrote: > It might be less confusing to just have one. Hasn't worked so far ;) Well we're talking about changing the name from OnUpdateInt to OnUpdateLong. It shou

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

In message <[EMAIL PROTECTED]> on Wed, Mar 05, 2003 at 07:24:20AM +0200, Andi Gutmans wrote: > It might be less confusing to just have one. Hasn't worked so far ;) -- PHP Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

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

At 11:03 PM 3/4/2003 +0100, Sascha Schumann wrote: On Tue, 4 Mar 2003, Jani Taskinen wrote: > > Yup, that was the idea. I'll first change them > all to OnUpdateInteger, and then use your patch > to change the ones that need to be long to use OnUpdateLong. Is there any specific reas

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

On Tue, 4 Mar 2003, David Hill wrote: > > >> Is there any specific reason why a single API (OnUpdateLong) >> is not sufficient? Is not it a safe assumption that those >> modules which still use 'int's are simply the result of a >> mistake on the developer's side? > >This is a reas

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

> Is there any specific reason why a single API (OnUpdateLong) > is not sufficient? Is not it a safe assumption that those > modules which still use 'int's are simply the result of a > mistake on the developer's side? This is a reasonable assumption actually I can't think of

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

On Tue, 4 Mar 2003, Jani Taskinen wrote: > > Yup, that was the idea. I'll first change them > all to OnUpdateInteger, and then use your patch > to change the ones that need to be long to use OnUpdateLong. Is there any specific reason why a single API (OnUpdateLong) is not suff

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

Yup, that was the idea. I'll first change them all to OnUpdateInteger, and then use your patch to change the ones that need to be long to use OnUpdateLong. --Jani On Tue, 4 Mar 2003, David Hill wrote: > > >The patch I sent in should have all of the OnUpdateLong changes

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

The patch I sent in should have all of the OnUpdateLong changes in them, and the remaining OnUpdateInt should probably be OnUpDateInteger. The painful part was actually looking at what was passed to match up the calls. That might save you some time. dave hill > > Yes, I know, but we need t

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

And of course the rest to use OnUpdateInteger().. --Jani On Tue, 4 Mar 2003, Jani Taskinen wrote: > >Yes, I know, but we need to remain backwards compatible >so I'm adding OnUpdateInteger() and OnUpdateLong(). > >This leaves OnUpdateInt() as it is now. Just need to >

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

Yes, I know, but we need to remain backwards compatible so I'm adding OnUpdateInteger() and OnUpdateLong(). This leaves OnUpdateInt() as it is now. Just need to go through the extensions and change the necessary OnUpdateInt()'s to OnUpdateLong(). --Jani On Tue, 4

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

> 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..? If you want to leave the current OnUpdateInt behavior (uses long * inte

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

> Don't they have to do that anyway..? :) No, why? For example, the session extension will be largely unchanged. The same code works in PHP 4 and 5. - Sascha -- PHP Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

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

On Tue, 4 Mar 2003, Sascha Schumann wrote: >> 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..? > >Yes. Ok, I'll

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

> 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..? Yes. > And in 5.0.0 we change OnUpdateInt() to use ints. No

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

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 >

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

At 17:50 28/02/2003, Sascha Schumann wrote: I think that simply adding OnUpdateLong and deprecating OnUpdateInt is fine while retaining its current semantics. +1 Zeev -- PHP Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

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

On Fri, 28 Feb 2003, Andi Gutmans wrote: > At 04:50 PM 2/28/2003 +0100, Sascha Schumann wrote: > > I think that simply adding OnUpdateLong and deprecating > > OnUpdateInt is fine while retaining its current semantics. I > > just don't see any value in changing the meaning of > > O

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

> That's also an option but I think OnUpdateInt() is confusing and how do we > stop ppl from using it in new extensions which who's commit messages aren't > followed via php-cvs? I volunteer to set up a cron job which greps for OnUpdateInt :-) The problem with _changing_ the existing

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

At 04:50 PM 2/28/2003 +0100, Sascha Schumann wrote: I think that simply adding OnUpdateLong and deprecating OnUpdateInt is fine while retaining its current semantics. I just don't see any value in changing the meaning of OnUpdateInt; at least that's how I interpreted Andi's mes

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

I think that simply adding OnUpdateLong and deprecating OnUpdateInt is fine while retaining its current semantics. I just don't see any value in changing the meaning of OnUpdateInt; at least that's how I interpreted Andi's message. - Sascha -- PHP Development Mailing Lis

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

At 04:47 PM 2/28/2003 +0100, Sascha Schumann wrote: > 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. Well, and how are you planning to mitigate the BC issues? Remember that not all exte

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

> 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. Well, and how are you planning to mitigate the BC issues? Remember that not all extension code is under our direct control. -

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

> > 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? > > - Sascha Most, but not al

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

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 incor

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

> 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? - Sascha -- PHP Development Mailing L

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

Hi, The reason the zend_parse_parameters() API only exposes long is because the Zend zval (container which holds the engine values) only supports longs and not ints. So anything which touches scripting engine values should be using long. However, extensions which have INI parameters may want to

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

Hi, Preface: This e-mail uses the distribution list (To and CC addresses) that the original poster used. I have moved everyone other than php-dev to Bcc so that this doesn't get perpetuated (I have no idea of the significance of the other addresses that were e-mailed, nor do I know Dave Hill's sta

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

Proposed Patch to address 64 bit issues in PHP v4.3.1 Diff -u against php4-STABLE-200302241430 Also needed in 4.5.x and 5.x Addresses bugs 20994, 21002, 21822, 20268 Platforms affected, Tru64, Solaris 9, HP-UX, NetBSD/Alpha and any other 64 bit platform I forgot to mention A little education f