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

Support await expression in yield expression #720

Closed
dsherret opened this issue Mar 14, 2020 · 1 comment · Fixed by #727
Closed

Support await expression in yield expression #720

dsherret opened this issue Mar 14, 2020 · 1 comment · Fixed by #727
Assignees
Labels
Milestone

Comments

@dsherret
Copy link
Contributor

Input code

async function* main() {
  yield await 0;
}

Expected behavior

Parses.

Actual behaviour

Line 2, column 9: Expected ';', '}' or <eof>

Version

master

I looked into this a bit, but I wasn't sure why it was parsing an assignment expression for a yield expression's expression (I'm not too familiar with this).

@dsherret dsherret added the C-bug label Mar 14, 2020
@kdy1 kdy1 added this to the v1.1.36 milestone Mar 16, 2020
@kdy1 kdy1 self-assigned this Mar 25, 2020
kdy1 added a commit to kdy1/swc that referenced this issue Mar 25, 2020
@kdy1 kdy1 mentioned this issue Mar 25, 2020
@kdy1 kdy1 closed this as completed in #727 Mar 25, 2020
kdy1 added a commit that referenced this issue Mar 25, 2020
 - Allow await in an yield expression (fixes #720)
 - Prevent duplicate tokens while capturing (fixes #726)
@swc-bot
Copy link
Collaborator

swc-bot commented Oct 28, 2022

This closed issue has been automatically locked because it had no new activity for a month. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.

@swc-project swc-project locked as resolved and limited conversation to collaborators Oct 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging a pull request may close this issue.

3 participants