Re: Re: svn: /php/php-src/ branches/PHP_5_2/Zend/tests/bug51421.phpt branches/PHP_5_2/Zend/zend_compile.c branches/PHP_5_3/Zend/tests/bug51421.phpt
branches/PHP_5_3/Zend/zend_compile.c trunk/Zend/tests/bug51421.phpt trunk/Zend/zen
| From: | Derick Rethans | Date: | Mon, 28 Jun 2010 09:42:18 +0000 |
| Subject: | Re: Re: svn: /php/php-src/ branches/PHP_5_2/Zend/tests/bug51421.phpt branches/PHP_5_2/Zend/zend_compile.c branches/PHP_5_3/Zend/tests/bug51421.phpt branches/PHP_5_3/Zend/zend_compile.c trunk/Zend/tests/bug51421.phpt trunk/Zend/zen |
||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-48978@lists.php.net to get a copy of this message | ||
On Mon, 28 Jun 2010, Pierre Joye wrote:
> 2010/6/28 Johannes Schlüter <johannes@schlueters.de>:
> >
> > On Sat, 2010-06-26 at 22:05 +0000, Felipe Pena wrote:
> >>
> >> Log:
> >> - Fixed bug #51421 (Abstract __construct constructor argument
> >> list not enforced)
> >>
> >> Bug: http://bugs.php.net/51421 (Closed) Abstract
> >> __construct
> >> constructor argument list not enforced
> >
> > Won't this break compatibility?
> >
> > I'd say the change is logically correct but not sure we should break BC
> > there during RC.
>
> Did we not discuss this problem recently? If I'm not mistaken the
> decision was to do not change the way constructors work in 5.x.
That's what I seem to remember as well.
Derick
--
http://derickrethans.nl | http://xdebug.org
Like Xdebug? Consider a donation: http://xdebug.org/donate.php
twitter: @derickr and @xdebug