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

enable unsafe for test/ufuzz.js #2819

Merged
merged 1 commit into from Jan 19, 2018
Merged

enable unsafe for test/ufuzz.js #2819

merged 1 commit into from Jan 19, 2018

Conversation

alexlamsl
Copy link
Collaborator

  • introduce unsafe_undefined
  • safer .toString() compression

Miscellaneous

  • rename unsafe_Function

@alexlamsl
enable unsafe for test/ufuzz.js
20bfd05 
- introduce `unsafe_undefined`
- safer `.toString()` compression

Miscellaneous
- rename `unsafe_Function`
@alexlamsl
Copy link
Collaborator Author

@kzc after this, we can contemplate on whether to extend unsafe_undefined to handle #2585

@alexlamsl alexlamsl merged commit 069df27 into mishoo:master Jan 19, 2018
@alexlamsl alexlamsl deleted the unsafe branch January 19, 2018 15:47
@alexlamsl
Copy link
Collaborator Author

unsafe_math strikes: https://travis-ci.org/alexlamsl/UglifyJS2/builds/331147350

original result:
null 3164.93234476367 1.9323447636700648 7098 Infinity NaN undefined

uglified result:
null 3164.9323447636702 1.9323447636700648 7098 Infinity NaN undefined
Suspicious compress options:
  evaluate
  unsafe_math

Seems rare and obvious enough to be kept around.

@alexlamsl
Copy link
Collaborator Author

Another unsafe_math: https://travis-ci.org/alexlamsl/UglifyJS2/builds/331324908

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@alexlamsl