Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Apparently obsolete warning about finally not being called when yielding rejected promises from a sync generator #37038

Open
Hexstream opened this issue Dec 1, 2024 · 3 comments
Labels
Content:JS JavaScript docs needs BCD update waiting for implementations Waiting for feature to be implemented in browsers

Comments

@Hexstream
Copy link

Hexstream commented Dec 1, 2024

MDN URL

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/for-await...of

What specific section or headline is this issue about?

From Note: until but not including Specification.

What information was incorrect, unhelpful, or incomplete?

It seems at least modern Chrome does correctly call finally in the specified circumstances.

Assuming this is the case in all modern browsers and is perhaps even specified as such,
the corresponding content should be removed.

What did you expect to see?

No warning about an obsolete issue, assuming it actually is.

Do you have any supporting links, references, or citations?

Someone should check what the relevant spec says, I'm too busy, sorry.

Do you have anything more you want to share?

No response

MDN metadata

Page report details
@Hexstream Hexstream added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Dec 1, 2024
@github-actions github-actions bot added the Content:JS JavaScript docs label Dec 1, 2024
@Josh-Cena
Copy link
Member

This is a very recent change whose spec isn't even merged: tc39/ecma262#2600 V8 issue fixed in April, but neither of the two other engines have caught up. We need to wait some more, and this would need compat data.

@Josh-Cena Josh-Cena added waiting for implementations Waiting for feature to be implemented in browsers needs BCD update and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Dec 2, 2024
@vzexadel
Copy link

It's still an issue at least in firefox
Currently v133

@vzexadel
Copy link

nodejs has this issue being fixed only for v23 which is not LTS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:JS JavaScript docs needs BCD update waiting for implementations Waiting for feature to be implemented in browsers
Projects
None yet
Development

No branches or pull requests

3 participants