Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:49643
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain nebm.ist.utl.pt from 193.136.128.22 cause and error)
Content-Type: text/plain; charset=iso-8859-15; format=flowed; delsp=yes
To: internals@lists.php.net
References: <AANLkTimuWDV=9AA63_9M6cQSp9riCPE214sUNdoV3tQn@mail.gmail.com>
 <4C879613.7090709@zend.com>
 <AANLkTim-dHYyJxX2R3-PBVqqa3sv3+zKYr7zw7QvyngJ@mail.gmail.com>
 <4C887D2B.2000605@zend.com> <4C8AC526.7000505@sugarcrm.com>
 <AANLkTikpCGj2zVNQmC1gUWaRSpsQD=qNrETi90wj_No7@mail.gmail.com>
 <4C8B6168.30504@mohiva.com>
 <AANLkTinuD_Aga-CbsbSq-23MTyOLPGrBEswTm=JtBY0H@mail.gmail.com>
 <4C8BC81E.8000605@sugarcrm.com>
 <AANLkTinFSqCFUE3P5XvjJPwUGZqYrQR_9VHAXAWSL=rv@mail.gmail.com>
 <7.0.1.0.2.20100913145703.0d226d90@zend.com>
 <AANLkTi=KCpgNwhq55+f9+60K=y197HTbifaAd7=9qPop@mail.gmail.com>
 <7.0.1.0.2.20100913172559.0d2277d0@zend.com>
 <op.vizacsuhidpuyk@cataphract-old.dulce.lo.geleia.net>
 <7.0.1.0.2.20100913183427.0d2277d0@zend.com>
 <op.vizeorx8idpuyk@cataphract-old.dulce.lo.geleia.net>
 <4C8E6B6F.4080308@sugarcrm.com>
 <op.vizk9rxvidpuyk@cataphract-old.dulce.lo.geleia.net>
 <op.viznfkg1idpuyk@cataphract-old.dulce.lo.geleia.net>
 <4C8E8ED4.9040502@sugarcrm.com>
Date: Mon, 13 Sep 2010 22:53:13 +0100
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Organization: =?iso-8859-15?Q?N=FAcleo_de_Eng=2E_Biom=E9dica_?=
 =?iso-8859-15?Q?do_IST?=
Message-ID: <op.vizq2zb0idpuyk@cataphract-old.dulce.lo.geleia.net>
In-Reply-To: <4C8E8ED4.9040502@sugarcrm.com>
User-Agent: Opera Mail/10.61 (Win32)
Subject: Re: [PHP-DEV] Re: PHP Annotations RFC + Patch
From: glopes@nebm.ist.utl.pt ("Gustavo Lopes")

On Mon, 13 Sep 2010 21:51:32 +0100, Stas Malyshev <smalyshev@sugarcrm.com>  
wrote:

>> If I'm not mistaken, the current implementation instantiates an object
>> each time getAnnotation() is called, but it was proposed to change this
>> into a lazy-loading mechanism with the same instance returned every time
>> for each annotation. In that case, we'd only need to validate that one
>> time.

(What I describe as "proposed" is actually the implemented behavior)

>
> Object instances can't be bytecode-cached, so when are they  
> instantiated? Does it mean it's stateful and mutable?

You're confusing multiple calls to the script to multiple calls to  
getAnnotation().
You're right in that one cannot keep objects between requests. The  
annotation will have to be regenerated, and possibly validated, on each  
script call. It's indeed unfortunate, but we could add the possibility of  
turning off these verifications in production or add a hook that  
extensions could use so they could cache the result of the validation.  
It's not a show-stopper.

However, this does not equal instantiations for each call to  
getAnnotation(). This only creates one object:

class A extends ReflectionAnnotation {}

[A]
class H { }

$refH = new ReflectionClass("H");
$refH->getAnnotations();
$refH->getAnnotations();


-- 
Gustavo Lopes