Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:57406
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.21 cause and error)
Content-Type: text/plain; charset=utf-8; format=flowed; delsp=yes
To: internals@lists.php.net
References: <CAM4sM9FhzFv70U-X8-iRC52_0c+2=_gbVOjjx9_g8d5Dux-rHQ@mail.gmail.com> <op.v78y7hnxidpuyk@damnation> <4F15F47E.6030501@sugarcrm.com> <CAM4sM9EquFEQH0GcgjAfY8cREEk+mrGTv4dKoANNXMm34cVq6g@mail.gmail.com>
Date: Wed, 18 Jan 2012 11:35:37 +0100
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Organization: =?utf-8?Q?N=C3=BAcleo_de_Eng=2E_Biom=C3=A9di?=
 =?utf-8?Q?ca_do_I=2ES=2ET=2E?=
Message-ID: <op.v79zpnsxidpuyk@slws007.slhq.int>
In-Reply-To: <CAM4sM9EquFEQH0GcgjAfY8cREEk+mrGTv4dKoANNXMm34cVq6g@mail.gmail.com>
User-Agent: Opera Mail/11.60 (Win32)
Subject: Re: [PHP-DEV] Reflection to remove final
From: glopes@nebm.ist.utl.pt ("Gustavo Lopes")

On Wed, 18 Jan 2012 00:00:09 +0100, Jan Dolecek <juzna.cz@gmail.com> wrote:

> This issue seems much more complicated than I thought. We'll need to
> consider all cases which could cause troubles and have a solution for
> them.
>
> Namely:
>  - rewriting permanent structures for internal classes, which are
> being kept between requests, must be avoided
>  - races in threading models
>  - rewriting in bytecode caches must be avoided
>  - anything else in mind?
>
> I'll try to think about it, but I'm not that experienced with php
> internals, so any suggestions welcome.
>

I think you should approach it in a different way. Don't remove the final  
modifier, just ignore it in some circumstances (e.g. interface,  
annotation). PHP doesn't support annotations (at the engine level) and  
interface processing is currently done after checking the final class  
flag; additionally, there may be some code that relies final classes not  
having subclasses to this is probably still not going to be trivial.

-- 
Gustavo Lopes