RESOLVED FIXED268271
[JSC] Array destructuring assignment should close the iterator if PutValue or initializer evaluation throws 
https://bugs.webkit.org/show_bug.cgi?id=268271
Summary [JSC] Array destructuring assignment should close the iterator if PutValue or...
Alexey Shvayka
Reported 2024-01-28 21:26:30 PST
Related test262 failures: * test/language/expressions/assignment/dstr/array-elem-iter-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-elem-iter-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-elem-iter-rtrn-close.js * test/language/expressions/assignment/dstr/array-elem-iter-thrw-close-err.js * test/language/expressions/assignment/dstr/array-elem-iter-thrw-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-rtrn-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-thrw-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-thrw-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-rtrn-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-thrw-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-thrw-close.js * test/language/expressions/assignment/dstr/array-rest-iter-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-rest-iter-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-rest-iter-rtrn-close.js * test/language/expressions/assignment/dstr/array-rest-iter-thrw-close-err.js * test/language/expressions/assignment/dstr/array-rest-iter-thrw-close.js * test/language/expressions/assignment/dstr/array-rest-lref-err.js * test/language/statements/for-of/dstr/array-elem-iter-rtrn-close-err.js * test/language/statements/for-of/dstr/array-elem-iter-rtrn-close-null.js * test/language/statements/for-of/dstr/array-elem-iter-rtrn-close.js * test/language/statements/for-of/dstr/array-elem-iter-thrw-close-err.js * test/language/statements/for-of/dstr/array-elem-iter-thrw-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-rtrn-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-rtrn-close-null.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-rtrn-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-thrw-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-thrw-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-rtrn-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-rtrn-close-null.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-rtrn-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-thrw-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-thrw-close.js * test/language/statements/for-of/dstr/array-rest-iter-rtrn-close-err.js * test/language/statements/for-of/dstr/array-rest-iter-rtrn-close-null.js * test/language/statements/for-of/dstr/array-rest-iter-rtrn-close.js * test/language/statements/for-of/dstr/array-rest-iter-thrw-close-err.js * test/language/statements/for-of/dstr/array-rest-iter-thrw-close.js * test/language/statements/for-of/dstr/array-rest-lref-err.js
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2024-01-30 14:38:49 PST
<rdar://problem/121960887>
Alexey Shvayka
Comment 2 2024-08-27 14:11:56 PDT
Pull request: https://github.com/WebKit/WebKit/pull/32778
EWS
Comment 3 2024-09-10 15:33:17 PDT
Committed 283443@main (863e4a5d13af): <https://commits.webkit.org/283443@main> Reviewed commits have been landed. Closing PR #32778 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.