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.

Sign up for GitHub

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

Jump to bottom

Build optimization breaks code (Removes necessary break from switch) #4930

Closed
DeepDoge opened this issue Apr 8, 2023 · 1 comment · Fixed by #4937
Closed

Build optimization breaks code (Removes necessary break from switch) #4930

DeepDoge opened this issue Apr 8, 2023 · 1 comment · Fixed by #4937
Labels
t¹ 🐞 bug t⁸ ⋅ triage

Comments

@DeepDoge
Copy link

DeepDoge commented Apr 8, 2023 •
edited

Rollup Version

3.20.2

Operating System (or Browser)

Rollup REPL

Node Version (if applicable)

No response

Link To Reproduction

Rollup REPL

Expected Behaviour

break keyword shouldn't be removed and "error 3" shouldn't be thrown.

Actual Behaviour

break keyword is being removed on build and that causes "error 3" to be thrown
@DeepDoge DeepDoge mentioned this issue Apr 8, 2023
Closed
7 tasks
@TrickyPi TrickyPi mentioned this issue Apr 13, 2023
Merged
9 tasks
@rollup-bot
Copy link
Collaborator

This issue has been resolved via #4937 as part of rollup@3.20.4. You can test it via npm install rollup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
t¹ 🐞 bug t⁸ ⋅ triage
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: handle conditional breaks in nested switch statement cases TrickyPi/rollup
2 participants
@DeepDoge @rollup-bot