Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:40417
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain chiaraquartet.net from 208.83.222.18 cause and error)
Message-ID: <48C6B17A.90307@chiaraquartet.net>
Date: Tue, 09 Sep 2008 12:25:14 -0500
User-Agent: Thunderbird 2.0.0.6 (Macintosh/20070807)
MIME-Version: 1.0
To: Stanislav Malyshev <stas@zend.com>
CC:  internals@lists.php.net
References: <48C59D5C.4050507@chiaraquartet.net> <48C5A909.4030502@zend.com> <48C602D5.6020704@chiaraquartet.net> <48C6A6AA.7050003@zend.com>
In-Reply-To: <48C6A6AA.7050003@zend.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Subject: Re: [PATCH] Re: [PHP-DEV] namespace examples (solving name resolution
 order issues)
From: greg@chiaraquartet.net (Greg Beaver)

Stanislav Malyshev wrote:
> Hi!
>
>> a use statement.  For those who aren't clear on what Stas is saying,
>> here is the better example:
>>
>> <?php
>> namespace PEAR2::Pyrus::PackageFile::v2Iterator;
>> use namespace::File, namespace::FileAttribsFilter,
>> namespace::FileContents;
>> ?>
>
> No, actually that's not what I meant. You can do this too, but I meant
> just use namespace::File. There's no law about using :: in the code,
> really. Just write namespace::File, it's both short and clear.
>
>> Now for the good news.  The reason it is so slow is that the autoload is
>> called for every single invocation of the class name.  With the patch at
>> http://pear.php.net/~greg/resolution_fix.patch.txt this performance is
>
> What this patch does?
>
>> Now that the performance hit is minimal, can we agree to get along on
>> this proposed change Stas? ;)  It transforms the critical nature of the
>> problem to a truly minor performance hit on internal class usage without
>> use or ::.
>
> Please read my response to your previous email, I have described there
> issues with your proposal and how to make it non-contradictory and
> what else needs to be done.

Stas,

I did read your message and posted a response, it is you who needs to
read my reply.  "What this patch does?" is patent evidence that you did
not.  Please try again.

Greg