RE: [ZEND-ENGINE-CVS] cvs: ZendEngine2(PHP_5_1) /zend_compile.c

From: Date: Fri, 18 Nov 2005 19:34:26 +0000
Subject: RE: [ZEND-ENGINE-CVS] cvs: ZendEngine2(PHP_5_1) /zend_compile.c
References: 1 2  Groups: php.zend-engine.cvs 
Request: Send a blank email to [email protected] to get a copy of this message 
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:[email protected]]
Sent: Friday, November 18, 2005 4:10 AM
To: Derick Rethans; Dmitry Stogov
Cc: [email protected]
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!

Thread (19 messages)

« previous php.zend-engine.cvs (#4365) next »