[JSC] Update iterator built-ins to reflect the change of github:tc39/ecma262#3776#65174
Conversation
|
EWS run on previous version of this PR (hash f7a97e4) Details |
|
test262 still not contains the test of tc39/ecma262#3776 yet... |
f7a97e4 to
ab8c522
Compare
|
EWS run on previous version of this PR (hash ab8c522) Details |
|
EWS run on previous version of this PR (hash 1c859ae) Details |
1c859ae to
8167400
Compare
|
EWS run on previous version of this PR (hash 8167400) Details |
I opened tc39/test262#5059 |
8167400 to
4a33091
Compare
|
EWS run on previous version of this PR (hash 4a33091) Details |
4a33091 to
4c3cf85
Compare
|
EWS run on previous version of this PR (hash 4c3cf85) Details |
4c3cf85 to
b3d173d
Compare
|
EWS run on previous version of this PR (hash b3d173d) Details |
b3d173d to
fdb57de
Compare
|
EWS run on previous version of this PR (hash fdb57de) Details |
fdb57de to
21c29a4
Compare
|
EWS run on current version of this PR (hash 21c29a4) Details |
…ecma262#3776 https://bugs.webkit.org/show_bug.cgi?id=315085 Reviewed by Yusuke Suzuki. The change of tc39/ecma262#3776 for iterator helper builtins are categorized into: - Our implementations should be changed due to they should throw a `RangeError` corresponds to the user input value. - `Iterator.prototype.drop` - `Iterator.prototype.take` - The semantics has been a bit changed but it's not related to our implementation actually. - `Iterator.prototype.filter` - `Iterator.prototype.find` - `Iterator.prototype.flatMap` - `Iterator.prototype.forEach` - `Iterator.prototype.map` - `Iterator.prototype.reduce` - `Iterator.prototype.some` Tests: JSTests/stress/iterator-prototype-drop-bug315085.js JSTests/stress/iterator-prototype-take-bug315085.js * JSTests/stress/iterator-prototype-drop-bug315085.js: Added. https://commits.webkit.org/284597@main implements `Iterator.prototype.drop` but no stress tests. This change adds a minimum testcase. * JSTests/stress/iterator-prototype-take-bug315085.js: Added. https://commits.webkit.org/284597@main implements `Iterator.prototype.take` but no stress tests. This change adds a minimum testcase. * Source/JavaScriptCore/builtins/JSIteratorPrototype.js: Canonical link: https://commits.webkit.org/315276@main
21c29a4 to
8310ec8
Compare
|
Committed 315276@main (8310ec8): https://commits.webkit.org/315276@main Reviewed commits have been landed. Closing PR #65174 and removing active labels. |
🛠 win
8310ec8
21c29a4
🛠 playstation