On Tue, Jul 8, 2008 at 3:39 PM, Dmitry Stogov <[EMAIL PROTECTED]> wrote: > 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?
"class names" don't because, people expect their results to be context-independent. It is just a call of "some method" on "some class". "parent", on the other hand, implies "inheritance". self and static should do forwarding too, I think, because they are context-dependent by design. forwardind is expected here > > 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. -- Alexey Zakhlestin http://blog.milkfarmsoft.com/ -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
