Skip to content
This repository has been archived by the owner on Jan 26, 2022. It is now read-only.

Access the .next method only once as per https://github.com/tc39/ecma262/pull/988 #116

Closed
Jamesernator opened this issue Oct 26, 2017 · 1 comment
Closed

Access the .next method only once as per https://github.com/tc39/ecma262/pull/988 #116

Jamesernator opened this issue Oct 26, 2017 · 1 comment

Comments

@Jamesernator
Copy link

The tests for this seem to be already merged into Test262 (here and here) so the spec text here should probably also be changed to reflect the changed behavior.
@domenic
Copy link
Member

domenic commented Oct 26, 2017

Yep, that's the plan, just haven't had the time yet. It's a bit tricky since a lot of the underlying spec machinery has changed out from under this proposal.

domenic added a commit that referenced this issue Dec 4, 2017
@domenic
Align with the base spec in caching "next" methods
36c46be 
Closes #116. See tc39/ecma262#1021.
@domenic domenic mentioned this issue Dec 4, 2017
Merged
domenic added a commit that referenced this issue Dec 6, 2017
@domenic
Align with the base spec in caching "next" methods
bf36f72 
Closes #116. See tc39/ecma262#1021.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@domenic @Jamesernator