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

2002-12-12 Thread Zeev Suraski
[Stas answered you in a very thorough and accurate way, and since it took several hours since I started composing this reply and the time I actually sent it, I cut it down to minimum] Marcus, I (as well as basically all major OO languages I can think of) disagree with you. OO and inheritance a

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

2002-12-12 Thread Stanislav Malyshev
MB>> 1) Visibility is about information hiding not showing. Visibility is both hiding and showing. Visibility is defining what is shown and what is hidden. MB>> 2) You are wrong about 'is-a' in general. Our main problem here is that we MB>> do not have a typecast operator. I do not see wh

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

2002-12-12 Thread Marcus Börger
Since more feedback may help here i also send this to php-dev At 15:47 08.12.2002, Stanislav Malyshev wrote: 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 "

[PHP-DEV] Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend_compile.c zend_compile.h zend_execute.c zend_language_parser.y zend_language_scanner.l

2002-11-21 Thread Marcus Börger
Tried a bit with your implementation and i wonder why you allow redeclared functions to be abstract: class pass { function show() { echo "Call to function show()\n"; } } class fail extends pass { abstract function show(); } Another thing i wonder is that y

[PHP-DEV] Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend_compile.c zend_compile.h zend_execute.c zend_language_parser.y

2002-09-24 Thread Sebastian Bergmann
Andi Gutmans wrote: > andiTue Sep 24 15:05:53 2002 EDT > > Modified files: > /ZendEngine2zend_compile.c zend_compile.h zend_execute.c > zend_language_parser.y > Log: > - Megapatch to try and support inheritance from sub-classes. Things > m

[PHP-DEV] Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend_compile.c zend_compile.h zend_execute.c zend_language_parser.y zend_language_scanner.l zend_operators.c zend_operators.h

2002-08-03 Thread Andi Gutmans
At 02:43 AM 7/30/2002 -0400, Andrei Zmievski wrote: >On Tue, 30 Jul 2002, Andi Gutmans wrote: > > Hey, > > > > Can't you share the patch before you just go ahead and just commit it? I > > still have the same issue I used to have. > >Hey, the patch has been available for a long while now. :-) > > >