RE: [ZEND-ENGINE-CVS] 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@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 @ <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!