RE: [ZEND-ENGINE-CVS] cvs: ZendEngine2(PHP_5_1) /zend_compile.c
| From: | Dmitry Stogov | Date: | Fri, 18 Nov 2005 07:51:24 +0000 |
| Subject: | RE: [ZEND-ENGINE-CVS] cvs: ZendEngine2(PHP_5_1) /zend_compile.c | ||
| References: | 1 | Groups: | php.zend-engine.cvs |
| Request: | Send a blank email to php-zend-engine-cvs+get-4360@lists.php.net to get a copy of this message | ||
I don't see any reason not allow null as default value for arguments with
array type hint.
In real OOP languages "array" just a special class.
The situation before this patch was buggy.
Compiler allowed (array $a = null), but executor emited error.
Thanks. Dmitry.
> -----Original Message-----
> From: Andi Gutmans [mailto:andi@zend.com]
> Sent: Friday, November 18, 2005 4:10 AM
> To: Derick Rethans; Dmitry Stogov
> Cc: zend-engine-cvs@lists.php.net
> Subject: Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2(PHP_5_1)
> /zend_compile.c
>
>
> Yeah. I don't see why null should be allowed either.
>
> At 10:55 AM 11/15/2005, Derick Rethans wrote:
> >On Tue, 15 Nov 2005, Dmitry Stogov wrote:
> >
> > > dmitry Tue Nov 15 11:49:44 2005 EDT
> > >
> > > Modified files: (Branch: PHP_5_1)
> > > /ZendEngine2 zend_compile.c
> > > Log:
> > > Allow "null" default value for arguments with array type hint
> >
> >Hello Dmitry,
> >
> >I didn't implement this because in those cases you should set the
> >default to = array(), and this is working fine even before
> your patch:
> >
> >function boo(array $ar = array())
> >{
> >}
> >
> >This is why I didn't implement this for array, as it doesn't
> make much
> >sense to me.
> >
> >Derick
> >
> >--
> >Zend Engine CVS Mailing List (http://cvs.php.net/)
> >To unsubscribe, visit: http://www.php.net/unsub.php
>
> --
> Zend Engine CVS Mailing List (http://cvs.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>