Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:35092
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.146.183 as permitted sender)
DomainKey-Status: bad
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=unArvcp+Xe0eWLX8TNqHuT3TH7c6cs4kFW23zDMlM3K8fNkoT2yZYOyIAQsMhEVp6NWmqWizikRvy/AajrdrcncGi24PN4QLi7nJCN7nuLYAn7QbutdG+0o/clqTemDRcK0+5h6OnOeWa2WyOE83PpA3bOVAeyM4evUAfICY57Q=
Message-ID: <fe05d1540802011733k2c915fdeibf1ebeb0dc7ac65c@mail.gmail.com>
Date: Sat, 2 Feb 2008 02:33:26 +0100
To: "Gregory Beaver" <greg@chiaraquartet.net>
Cc: "Rasmus Lerdorf" <rasmus@lerdorf.com>, 
	"Cristian Rodriguez" <judas.iscariote@gmail.com>, 
	"Marcus Boerger" <helly@php.net>, internals@lists.php.net
In-Reply-To: <47A3BEB2.7090308@chiaraquartet.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <574269741.20080201224537@marcus-boerger.de>
	 <7d5a202f0802011436ga4cf3duc02f2354e88a0d5@mail.gmail.com>
	 <47A3A70B.20809@lerdorf.com> <47A3BEB2.7090308@chiaraquartet.net>
Subject: Re: [PHP-DEV] [ZEND-ENGINE-CVS] cvs: ZendEngine2(PHP_5_3) / zend_API.czend_API.h php-src/ext/standard type.c
From: pierre.php@gmail.com ("Pierre Joye")

On Feb 2, 2008 1:52 AM, Gregory Beaver <greg@chiaraquartet.net> wrote:
> Rasmus Lerdorf wrote:
> > Cristian Rodriguez wrote:
> >> 2008/2/1, Marcus Boerger <helly@php.net>:
> >>>   - Fix callable/static mess, the following will now all result in a E_STRICT
> >>>     . binding a dynamic function as a static callback
> >>>     . static call of a dynamic function
> >>>     . is_callable() on a static binding to a dynamic function
> >>
> >> Does not compile..
> >>
> >> php5.3/ext/spl/php_spl.c: In function 'zif_spl_autoload_register':
> >> php5.3/ext/spl/php_spl.c:423: error: too few arguments to function
> >> 'zend_is_callable_ex'
> >> php5.3/ext/spl/php_spl.c: In function 'zif_spl_autoload_unregister':
> >> php5.3/ext/spl/php_spl.c:515: error: too few arguments to function
> >> 'zend_is_callable_ex'
> >> make: *** [ext/spl/php_spl.lo] Error 1
> >
> > Did you update your Zend/ directory?  It compiles fine here.
> >
> > I've been working through this one with Marcus a bit, and even though he
> > fundamentally disagrees with the concept, he still wrote the code.  I
> > can't say I would have done the same in his place.  So thanks, Marcus.
>
> I and everyone in PEAR agrees, this was a very gallant move.  For those
> who are similarly grateful, Marcus does have a wishlist at
> http://pecl.php.net/wishlist.php/helly :)

To rectify a bit the shot, everyone agrees, but that's not the point :)

The initial move was actually wrong, that does not mean that Marcus is
the evil or does nothing good for PHP. That only means that it was a
bad move and should be corrected. We all do such moves once or two.
The problem is fixed now, thanks you.

However that does not answer my questions, what's the gain to make
such errors fatal?

--
Pierre
http://blog.thepimp.net | http://www.libgd.org