Skip to content

fixes #63#99

Merged
trullock merged 2 commits into
masterfrom
bug_63
Jun 8, 2020
Merged

fixes #63#99
trullock merged 2 commits into
masterfrom
bug_63

Conversation

@trullock
Copy link
Copy Markdown
Owner

@trullock trullock commented Jun 6, 2020

I'm not entirely sure if the outputted javascript is exactly "right", but given we're parsing mangled input markup I'm not sure what to do.

  1. This fixes the infinite loop, which even if the outputted JS is "wrong" at least it doesnt crash/hang
  2. What is NUglify's stance on handling bad input syntax - do we care, as long as errors are reported? Do we not output anything at all? Do we not output the mangled statement, and resume at the next good bit? This PR outputs something from the mangled input

@trullock trullock merged commit a13f95f into trullock:master Jun 8, 2020
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.

1 participant