Document the fact that Closure advanced optimizations are disabled #598
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello — this pull request documents the fact that these benchmarks disable Google Closure Compiler's "advanced optimizations." In my opinion this is important context for interpreting the results.
The specific context I intend to communicate in the pull request is:
If people are using these benchmarks to decide which minification library to use on their own JS projects, they should be aware of the Closure Compiler advanced optimizations possibility. Otherwise this set of benchmarks is misleading — or, at the very least, paints an incomplete picture.
I have no affiliation with Google or with the Closure Compiler project, aside from having used it for 10+ years now in my own projects. If you take the time to structure your code to take advantage of the "advanced optimizations," it results in significantly better minification than any other library.
Previously: issue #168 brought up the fact that these benchmarks don't use advanced optimizations, and the issue was closed. If the scope of these benchmarks is to remain on third-party JS libraries, I agree it makes sense not to do an advanced optimizations test — because that would require you to tweak those third-party JS libraries. But I do think you should be transparent about this limitation.