Re: parent:: forwarding
| From: | Dmitry Stogov | Date: | Tue, 08 Jul 2008 11:39:18 +0000 |
| Subject: | Re: parent:: forwarding | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-38844@lists.php.net to get a copy of this message | ||
Hi Etienne,
I took a look into the patch and I don't like it all.
At first, I don't see any consistency there.
Why parent:: does forwarding but self::, static:: and "class names" don't?
At second, it's too complicated.
I would propose more consistent (from my point of view) and simpler patch.
Each call to static method of parent class (it doesn't mater if it was
done using parent:: or something else) assumes forwarding of called context.
Thanks. Dmitry.
Etienne Kneuss wrote:
> Hi,
>
> The patch was made initially by Mike Lively, I adapted it and removed
> forward_static_call
>
>
> http://patches.colder.ch/Zend/lsb_parent_forwarding_53.patch?markup
>
> http://patches.colder.ch/Zend/lsb_parent_forwarding_HEAD.patch?markup
>
> They should merge without trouble.
>
> Regards
>