Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend_compile.c

MB>> Therefor i would vote for allowing only decreasing the visibility or force MB>> to keep it. Decreasing visibility contradicts the basic inheritance principle - that derived class "is-a" base class, i.e., basically, it can be used in any place the base class is used. If you restrict methods in derived class, it may lead to a situation where $foo->method() works with base class, but fails with derived class - which is, as I said, against the inheritance principle. As for increasing visibility - I don't see major problem in it. What could be the problem if derived class is allowed to open protected method to public? After all, it could proxy it anyways... -- Stanislav Malyshev, Zend Products Engineer stas@zend.com http://www.zend.com/ +972-3-6139665 ext.109