I don't think it's critical to include this patch, but I do think it'd be a good thing.
Do you really think it'll break BC for many applications? How many people have functions that use null(), false(), true()?


Andi

At 08:17 AM 7/16/2004 -0700, Sterling Hughes wrote:
oh, i didn't notice it at all, which i'll buy is probably my fault. :)

This  seems like its a bad optimization.  I believing in breaking BC
in some cases when its absolutely necessary, but an optimization like
this one could easily happen at the level of an optimizer, like APC or
Zend's product.  Why are we breaking BC for this?

-sterling

On Fri, 16 Jul 2004 09:55:57 +0200, Marcus Boerger <[EMAIL PROTECTED]> wrote:
> Hello Sterling,
>
> it was on internals and in TODO-5.1. Maybe not many people noticed
> it between discussing typehints and ifsetor.
>
> marcus
>
>
>
> Friday, July 16, 2004, 9:12:28 AM, you wrote:
>
> > woops, discussion should be on [EMAIL PROTECTED]
>
> > ---------- Forwarded message ----------
> > From: Sterling Hughes <[EMAIL PROTECTED]>
> > Date: Fri, 16 Jul 2004 00:11:53 -0700
> > Subject: Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 /
> > zend_language_parser.y zend_language_scanner.l
> > To: Marcus Boerger <[EMAIL PROTECTED]>
> > Cc: [EMAIL PROTECTED]
>
> > this breaks bc - it should at least be discussed....
>
> > -sterling
>
> > On Fri, 16 Jul 2004 06:50:58 -0000, Marcus Boerger <[EMAIL PROTECTED]> wrote:
> >> helly Fri Jul 16 02:50:58 2004 EDT
> >>
> >> Modified files:
> >> /ZendEngine2 zend_language_parser.y zend_language_scanner.l
> >> Log:
> >> - Speed up by making null/false/true reserved word which allows to drop
> >> an opcode (FETCH_CONSTANT) for every usage.
> >>
>
>


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

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



Reply via email to