On Tue, Feb 19, 2013 at 3:12 PM, Etienne Kneuss wrote:

On Tue, Feb 19, 2013 at 2:55 PM, Derick Rethans wrote:

Gah, no top posting!
On Tue, 19 Feb 2013, Etienne Kneuss wrote:
On Tue, Feb 19, 2013 at 2:25 PM, Derick Rethans wrote:
On Tue, 19 Feb 2013, Nikita Popov wrote:

This RFC proposes to remove the type-restrictions on Iterator keys
used in foreach:


I took over Levi's RFC and added a patch for it.
Under "Open Questions" you write:
What should be done with the keys that are valid in the iterator,
but not in the array? I think the best approach would be to just
set the array keys with the exact same semantics as PHP would do
(i.e. with all casts and warnings).
Would __toString be called in case the key was an object?
I think the warning can stay as-is, __toString is not necessarily
available so casting to string in all occasions is probably not what
we want.
I think it should cast to a string if possible. You are now making
iterator_to_array not work with the new feature, and I find that a bit
silly. Whether it should be __toString (or __toKey) I don't really care,
but this new support for objects should be supported everywhere (and
that includes iterator_to_array).
I think this RFC is orthogonal to iterator_to_array supporting conversions
from objects to keys.

$array[$obj] = 42; does not call __toString (or __toKey), it throws a
catchable fatal. IMO it would be inconsistent if iterator_to_array
gracefully accepted objects as keys.
I updated the patch to use the usual array assignment behavior. I.e. the
behavior will be the same as the following code:

function iterator_to_array($iter) {
foreach ($iter as $k => $v) {
$array[$k] = $v;
return $array;

I think it's reasonable to expect that the code will behave this way.

If there are no objections I'd like to vote on this already on February
28th or March 1st. It is a few days shy of the 2 week discussion period,
but I'd like to have this in 5.5 and the beta is tagged on the 7th already


Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupphp-internals @
postedFeb 25, '13 at 2:58p
activeFeb 25, '13 at 2:58p

1 user in discussion

Nikita Popov: 1 post



site design / logo © 2022 Grokbase