Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:39489
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain pooteeweet.org from 88.198.8.16 cause and error)
Cc: PHP Developers Mailing List <internals@lists.php.net>,
 =?ISO-8859-1?Q?Johannes_Schl=FCter?= <johannes@php.net>
Message-ID: <2673889B-5552-4CF6-AD95-4A8C63067E46@pooteeweet.org>
To: Dmitry Stogov <dmitry@zend.com>
In-Reply-To: <4890D5A2.7030904@zend.com>
Content-Type: text/plain; charset=US-ASCII; format=flowed; delsp=yes
Content-Transfer-Encoding: 7bit
Mime-Version: 1.0 (Apple Message framework v928.1)
Date: Wed, 30 Jul 2008 23:04:45 +0200
References: <AA8162A3-9C98-47E0-BB6D-D1A132501F36@pooteeweet.org> <4890D5A2.7030904@zend.com>
Subject: Re: [PHP-DEV] alpha1 status update
From: smith@pooteeweet.org (Lukas Kahwe Smith)


On 30.07.2008, at 22:57, Dmitry Stogov wrote:

>> - remove __toString() from closures (Johannes/Dmitry)
>
> What is wrong here?
> What behavior do you prefer?
> I just missed the discussion.

This was in the thread "closures questions" and later there were some  
votes by Stas and Marcus in favor of its removal in the "towards 5.3  
alpha1" thread.

Here is a comment by Christian about the topic:

>>> so do we even want the toString() method?
>> IMHO we should drop toString from Closure.
>
> There is one case which *may* cause problems:
>
> function doSomething ($callback) {
>  if (!is_callable ($callback)) {
>    throw new Exception (...);
>  }
>
>  // special treatment
>  if ($callback == 'default') {
>    ...
>  } else {
>    ...
>  }
> }
>
> Here, the comparison of $callback with 'default' will yield a notice
> that $callback couldn't be converted. The comparison will fail
> correctly, of course, but it could cause some confusion. The correct
> way of comparing would obviously be using ===, that won't cause these
> hickups.


regards,
Lukas Kahwe Smith
smith@pooteeweet.org