RE: cvs: ZendEngine2(PHP_5_1) /zend_compile.c

I can't believe I'm with Jani for a change :)

At 12:08 AM 11/18/2005, Jani Taskinen wrote:

>     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@xxxxxxxx]
> > > Sent: Friday, November 18, 2005 4:10 AM
> > > To: Derick Rethans; Dmitry Stogov
> > > Cc: zend-engine-cvs@xxxxxxxxxxxxx
> > > 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 @ <http://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