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

Panic with jsx element containing many spaces between attributes #614

Closed
dsherret opened this issue Jan 29, 2020 · 1 comment
Closed

Panic with jsx element containing many spaces between attributes #614

dsherret opened this issue Jan 29, 2020 · 1 comment
Labels
Milestone

Comments

@dsherret
Copy link
Contributor

Describe the bug

The following panic occurs:

thread 'jsx::reference::custom\issue-XXX\input.js' panicked at 'internal error: entered unreachable code: assertion failed: (span.start <= span.end).
 start = 24, end = 23', ecmascript\parser\src\parser\jsx.rs:90:27

Input code

const t = <test test  = "5" other =  {4}/>

Expected behavior

No panic even though this code is weird :)

Version

Latest.

Additional context

I see this is in jsx.rs parse_jsx_attr_value, but I don't see anything wrong in there so I gave up.

@dsherret dsherret added the C-bug label Jan 29, 2020
@kdy1 kdy1 added this to the v1.1.17 milestone Jan 29, 2020
kdy1 added a commit to kdy1/swc that referenced this issue Jan 29, 2020
bors bot pushed a commit that referenced this issue Jan 29, 2020
bors bot pushed a commit that referenced this issue Jan 29, 2020
@bors bors bot closed this as completed in 0165958 Jan 29, 2020
@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