Xuefer Tinys | 1 Feb 2005 05:00
Picon

Re: [PHP-DEV] Re: referencing Superglobals with variable variables inside functions

$_GET is solved at compile time, which is good, but this make other
variables bad at execution time?

how much does this patch slow execution down? it's one more hash lookup.
affect speed of $var or $$var? or both?

On Mon, 31 Jan 2005 14:24:41 -0800, Sara Golemon <pollita <at> php.net> wrote:
> > So in order for ZE to resolve the autoglobals correctly during runtime it
> > has to ask two questions for every *part* of every variable resolution:
> "Are
> > we checking against the active symbol table?  Is the index we're looking
> for
> > in the autoglobal registry?"  If so, replace active_symbol_table with the
> > global symbol_table, otherwise do the lookup as normal.
> >
> I knew this statement felt wrong when I wrote it so I dove in and looked at
> the code again....It's not actually necessary to test the what scope we're
> in because of the way the parser is laid out... however it does still add an
> extra hashtable lookup that shouldn't be necessary, so my original position
> stands as is for now.
> 
> If someone would like to benchmark the following patch (Or point out any
> fatal flaws in it) I'd be interrested in the results, but even if there's no
> perceptible difference over a million itterations, I'd probably still
> vote -1 on changing it.
> 
> -Sara
> 
> Index: Zend/zend_execute.c
> ===================================================================
(Continue reading)

M. Sokolewicz | 1 Feb 2005 11:41
Picon

Re: [PHP-DEV] Re: referencing Superglobals with variable variables inside functions

AFAICS, it should make every local-variable (non-global) lookup slower. 
Since it needs to do a lookup in the globals table first.

Xuefer Tinys wrote:
> $_GET is solved at compile time, which is good, but this make other
> variables bad at execution time?
> 
> how much does this patch slow execution down? it's one more hash lookup.
> affect speed of $var or $$var? or both?
> 
> 
> On Mon, 31 Jan 2005 14:24:41 -0800, Sara Golemon <pollita <at> php.net> wrote:
> 
>>>So in order for ZE to resolve the autoglobals correctly during runtime it
>>>has to ask two questions for every *part* of every variable resolution:
>>
>>"Are
>>
>>>we checking against the active symbol table?  Is the index we're looking
>>
>>for
>>
>>>in the autoglobal registry?"  If so, replace active_symbol_table with the
>>>global symbol_table, otherwise do the lookup as normal.
>>>
>>
>>I knew this statement felt wrong when I wrote it so I dove in and looked at
>>the code again....It's not actually necessary to test the what scope we're
>>in because of the way the parser is laid out... however it does still add an
>>extra hashtable lookup that shouldn't be necessary, so my original position
(Continue reading)

Sara Golemon | 1 Feb 2005 15:59
Picon

Re: [PHP-DEV] Re: referencing Superglobals with variable variables inside functions

>> how much does this patch slow execution down? it's one more hash lookup.
>> affect speed of $var or $$var? or both?
>>
> AFAICS, it should make every local-variable (non-global) lookup slower. 
> Since it needs to do a lookup in the globals table first.
>
Slows the global ones too, though that can be reduced (while slightly 
worsening the non-global lookups) by not bothering to lookup the auto_global 
table if we're in the global scope already...

-Sara 

--

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

taft | 1 Feb 2005 17:19

[PHP-DEV] For help

How can I write a module for PHP? Any ABC information?

--

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Andi Gutmans | 1 Feb 2005 20:26
Favicon

[PHP-DEV] PHP 5.1

Hey,

I just heard from Wez that PDO is in very advanced stages now (ready for beta).
I would like to start the PHP 5.1 release process. Due to the lack of 
testing both the new engine VM and PDO have received I would like to start 
with a beta process so that we get feedback.

I know there are still some fixes that need to be applied both in the 
engine and in extensions so I would like to release a beta on March 1st (a 
month from today).
Derick also mentioned that his new very much needed Date 
extension will be ready by that date.

I believe both PDO and Date should be included in the default distro. As 
far as PDO is concerned I think for each DB if it is selected at configure 
time, the relevant PDO extension should also be enabled. So doing 
--with-oci8 should enable both ext/oci8 and ext/pdo_oci. This will give 
users more of a choice, give more exposure to PDO which is one of the most 
important features of 5.1 and of course, it doesn't really cost us very 
much except for having to do some configure hacking.

Comments/Flames/Praises to this list :)

Andi

--

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

(Continue reading)

Marc Richards | 1 Feb 2005 20:58
Picon
Picon
Favicon

[PHP-DEV] Re: PHP 5.1


Andi Gutmans wrote:
> 
> Comments/Flames/Praises to this list :)
> 

All praise PHP 5.1.  All praise PDO.

Marc

--

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Stephan Schmidt | 1 Feb 2005 21:20

Re: [PHP-DEV] PHP 5.1

Hi,

Andi Gutmans schrieb:
> I believe both PDO and Date should be included in the default distro.
I'd like to see xmlreader be bundled as well. It's fast, easy to use and 
still very powerful.

Stephan
-- 
http://www.php-tools.net
http://www.schst.net
http://pear.php.net

--

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

George Schlossnagle | 1 Feb 2005 21:23
Favicon

Re: [PHP-DEV] PHP 5.1


On Feb 1, 2005, at 3:20 PM, Stephan Schmidt wrote:

> Hi,
>
> Andi Gutmans schrieb:
>> I believe both PDO and Date should be included in the default distro.
> I'd like to see xmlreader be bundled as well. It's fast, easy to use 
> and still very powerful.

+1

George

--

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Andi Gutmans | 1 Feb 2005 21:48
Favicon

Re: [PHP-DEV] PHP 5.1

Don't see a problem with that if it's release quality.

Andi

At 09:20 PM 2/1/2005 +0100, Stephan Schmidt wrote:
>Hi,
>
>Andi Gutmans schrieb:
>>I believe both PDO and Date should be included in the default distro.
>I'd like to see xmlreader be bundled as well. It's fast, easy to use and 
>still very powerful.
>
>Stephan
>--
>http://www.php-tools.net
>http://www.schst.net
>http://pear.php.net

--

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Rasmus Lerdorf | 1 Feb 2005 23:10

Re: [PHP-DEV] PHP 5.1

Andi Gutmans wrote:
> I believe both PDO and Date should be included in the default distro. As 
> far as PDO is concerned I think for each DB if it is selected at 
> configure time, the relevant PDO extension should also be enabled. So 
> doing --with-oci8 should enable both ext/oci8 and ext/pdo_oci. This will 
> give users more of a choice, give more exposure to PDO which is one of 
> the most important features of 5.1 and of course, it doesn't really cost 
> us very much except for having to do some configure hacking.

If I can get off my ass and get it finished up it would be good to get a 
default input filtering extension in there as well.  We have not done 
much to help people do proper input validation and although the hook for 
it is in 5.0 I doubt anybody has actually used it yet.  I'll try to get 
it into PECL in the next week or so for people to have a look.

-Rasmus

--

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php


Gmane