Brandon Benvie [:benvie] Reporter
	Description • 11 years ago

In the most recent ES6 spec (July 2013 revision) Maps and WeakMaps initialized with an iterable should attempt to call its "entries" method first, and then fallback to the iterator symbol (or "iterator" in our case since we're not using the Symbol yet).

See spec sections 15.14.1 and 15.15.1.

	Rick Waldron [:rwaldron]
	Comment 1 • 11 years ago

(In reply to Brandon Benvie [:bbenvie] from comment #0)
> In the most recent ES6 spec (July 2013 revision) Maps and WeakMaps
> initialized with an iterable should attempt to call its "entries" method
> first, and then fallback to the iterator symbol (or "iterator" in our case
> since we're not using the Symbol yet).
> 
> See spec sections 15.14.1 and 15.15.1.

WeakMap? Or Set? There is no entries method of WeakMap ;)

	Brandon Benvie [:benvie] Reporter
	Comment 2 • 11 years ago

(In reply to Rick Waldron from comment #1)
> WeakMap? Or Set? There is no entries method of WeakMap ;)

Sorry, the "its" there was unclear; that refers to the iterable passed to the constructor, not the initializing Map/WeakMap. But I think this bit of the spec has been changed recently to just use @@iterator, so this bug may be invalid anyway.

	Rick Waldron [:rwaldron]
	Comment 3 • 11 years ago

(In reply to Brandon Benvie [:bbenvie] from comment #2)
> (In reply to Rick Waldron from comment #1)
> > WeakMap? Or Set? There is no entries method of WeakMap ;)
> 
> Sorry, the "its" there was unclear; that refers to the iterable passed to
> the constructor, not the initializing Map/WeakMap. But I think this bit of
> the spec has been changed recently to just use @@iterator, so this bug may
> be invalid anyway.

Got it, thanks for clarifying

	Sankha Narayan Guria [:sankha]
	Comment 4 • 11 years ago

(In reply to Brandon Benvie [:bbenvie] from comment #2)

> Sorry, the "its" there was unclear; that refers to the iterable passed to
> the constructor, not the initializing Map/WeakMap. But I think this bit of
> the spec has been changed recently to just use @@iterator, so this bug may
> be invalid anyway.

According to https://people.mozilla.org/~jorendorff/es6-draft.html#sec-map-iterable-comparator you are right. So should we close this bug?

	Brandon Benvie [:benvie] Reporter
	Comment 5 • 11 years ago

Indeed.