Skip to content

Fix temp variable scoping in async generators#38121

Merged
rbuckton merged 1 commit into
masterfrom
fix37686
Apr 24, 2020
Merged

Fix temp variable scoping in async generators#38121
rbuckton merged 1 commit into
masterfrom
fix37686

Conversation

@rbuckton
Copy link
Copy Markdown
Contributor

@rbuckton rbuckton commented Apr 22, 2020

Fixes a bug where we were generating temporary variables in a downlevel async generator that collided with temporary variables from prior transformations. This was happening because we were treating the function in asyncGenerator(this, arguments, function () { ... }) as a temporary variable scope, however we should not reset temporary variables in this scope.

Fixes #37686
Fixes #37418
Supersedes #37484

@rbuckton rbuckton merged commit 38ff776 into master Apr 24, 2020
@rbuckton rbuckton deleted the fix37686 branch April 24, 2020 19:10
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

3 participants