Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:55938
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.213.42 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <op.v3u0gjv3idpuyk@clk-0081.mshome.net>
References: <CAEZPtU73tt_V4PKeevX5N3DCxo-r7bUnC1tEc+peaDc+uoMSVg@mail.gmail.com>
	<op.v3u0gjv3idpuyk@clk-0081.mshome.net>
Date: Mon, 24 Oct 2011 22:27:31 +0200
Message-ID: <CAEZPtU7z-ZqUQUjHaaXjFwtoiErxxyJ8XycqLN4qwzmgNQrRYQ@mail.gmail.com>
To: Gustavo Lopes <glopes@nebm.ist.utl.pt>
Cc: internals@lists.php.net, Etienne Kneuss <colder@php.net>
Content-Type: text/plain; charset=ISO-8859-1
Subject: Re: [PHP-DEV] BC break in 5.4, extending internal class
From: pierre.php@gmail.com (Pierre Joye)

hi,

On Mon, Oct 24, 2011 at 4:13 PM, Gustavo Lopes <glopes@nebm.ist.utl.pt> wrote:

> The change I did in 5.4 should be reverted and synced with 5.3 in the
> short term because it doesn't work reliably (even not considering the BC
> break) -- there are other methods for getting an invalid instance (the new
> reflection method and unserialize, at least).

Please do the revert, as you did the commit in the 1st place. As far
as I remember we have discussed this issue back then already as well.

> But even in 5.3, there are in SPL counter-measures that don't work
> reliably, like making a check in get_method.

Right, but that's another concern and the same cautiousness should be
taken there.

Thanks!
-- 
Pierre

@pierrejoye | http://blog.thepimp.net | http://www.libgd.org