RE: [ZEND-ENGINE-CVS] cvs: ZendEngine2(PHP_5_1) /zend_compile.c
| From: | Dmitry Stogov | Date: | Fri, 18 Nov 2005 08:20:19 +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-4362@lists.php.net to get a copy of this message | ||
May be not array but zval :)
Of course this is not for nearest future.
Dmitry.
> -----Original Message-----
> From: Jani Taskinen [mailto:sniper@iki.fi]
> Sent: Friday, November 18, 2005 11:08 AM
> To: Dmitry Stogov
> Cc: 'Andi Gutmans'; 'Derick Rethans'; zend-engine-cvs@lists.php.net
> Subject: RE: [ZEND-ENGINE-CVS] cvs: ZendEngine2(PHP_5_1)
> /zend_compile.c
>
>
>
> I can already see where this is going..next you're proposing
> that we change our dear old array into a class? :)
>
> --Jani
>
>
> On Fri, 18 Nov 2005, Dmitry Stogov wrote:
>
> >
> > 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
> >>
> >>
> >
> >
>
> --
> Give me your money at @ <'
> |#T ohttp://pecl.php.net/wishlist.php/sniper>
> Donating money may make me happier and friendlier for a
> limited period! Death to all 4 letter abbreviations starting with P!
>
> --
> Zend Engine CVS Mailing List (http://cvs.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>