Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:23913
Return-Path: <mike@php.net>
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Delivered-To: mailing list internals@lists.php.net
Received: (qmail 75157 invoked by uid 1010); 4 Jun 2006 10:07:14 -0000
Delivered-To: ezmlm-scan-internals@lists.php.net
Delivered-To: ezmlm-internals@lists.php.net
Received: (qmail 75136 invoked from network); 4 Jun 2006 10:07:14 -0000
Received: from unknown (HELO lists.php.net) (127.0.0.1)
  by localhost with SMTP; 4 Jun 2006 10:07:14 -0000
X-Host-Fingerprint: 80.123.98.46 unknown  
Received: from ([80.123.98.46:5520] helo=localhost.localdomain)
	by pb1.pair.com (ecelerity 2.0 beta r(6323M)) with SMTP
	id 29/90-49656-1D0B2844 for <internals@lists.php.net>; Sun, 04 Jun 2006 06:07:13 -0400
Message-ID: <29.90.49656.1D0B2844@pb1.pair.com>
To: internals@lists.php.net
Date: Sun, 04 Jun 2006 12:07:20 +0200
User-Agent: Thunderbird 1.5.0.2 (X11/20060522)
MIME-Version: 1.0
References: <795156743.20060603134212@marcus-boerger.de>
In-Reply-To: <795156743.20060603134212@marcus-boerger.de>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
X-Posted-By: 80.123.98.46
Subject: Re: Missing __toString() part
From: mike@php.net (Michael Wallner)

Marcus Boerger wrote:
> Hello guys,
> 
>   the attached patch closes one more __toString() part. It allows
> to use objects that define __toString as indexes to arrays. What do
> you guys think about this, should we add it or stay with the old
> behavior that didn't allow objects as indexes at all.

-1 on automatic __toString()
+1 on explicit (string) cast (as it is now)
+0 on __toHash()/__toKey() (which must be implemented explicitly)

Rasmus' idea of changing the HashKey is... duh... at least something
to think some more time about.

Regards,
-- 
Michael