Skip to content

Bugfix/loop destructuring#783

Merged
Perryvw merged 13 commits intomasterfrom
bugfix/loop-destructuring
Jan 2, 2020
Merged

Bugfix/loop destructuring#783
Perryvw merged 13 commits intomasterfrom
bugfix/loop-destructuring

Conversation

@Perryvw
Copy link
Copy Markdown
Member

@Perryvw Perryvw commented Dec 30, 2019

Fixed some issues regarding variable destructuring.

Fixes #770
Fixes #779

@Perryvw Perryvw requested a review from ark120202 December 30, 2019 18:03
ark120202 added a commit to ark120202/TypeScriptToLua that referenced this pull request Jan 1, 2020
Perryvw pushed a commit that referenced this pull request Jan 1, 2020
* Share scope between all switch cases

* Readd `do` blocks around cases and enable dead code elemination

* Use `pushScope` return value (from #783)

* Special-case switch scope hoisting

* Add test from PR feedback
@Perryvw Perryvw merged commit 8241018 into master Jan 2, 2020
@Perryvw Perryvw deleted the bugfix/loop-destructuring branch January 2, 2020 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Parameter destructuring leaks variables Complex destructuring patterns don't work in loops

2 participants