From fd5a76ac52f9fb5427c2b5a3311dfbed4510365a Mon Sep 17 00:00:00 2001 From: kzc Date: Thu, 30 Jun 2016 14:56:12 -0400 Subject: [PATCH] Change the default sequences limit to 200 to speed up compress. Has little or no impact on minification size in the majority of cases but can speed up rollup builds significantly. This sequences change also has the beneficial side effect of avoiding "stack size exceeded" errors on very large input files. The user is free to alter the sequences limit if they are so inclined. The previous sequences limit was 2000. 20 is often sufficient. --- README.md | 2 +- lib/compress.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index c0730874..721fa61b 100644 --- a/README.md +++ b/README.md @@ -292,7 +292,7 @@ to set `true`; it's effectively a shortcut for `foo=true`). - `sequences` (default: true) -- join consecutive simple statements using the comma operator. May be set to a positive integer to specify the maximum number of consecutive comma sequences that will be generated. If this option is set to - `true` then the default sequences limit is `2000`. Set option to `false` or `0` + `true` then the default sequences limit is `200`. Set option to `false` or `0` to disable. On rare occasions the default sequences limit leads to very slow compress times in which case a value of `20` or less is recommended. diff --git a/lib/compress.js b/lib/compress.js index 3da51abb..bd5e5f3a 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -80,7 +80,7 @@ function Compressor(options, false_by_default) { passes : 1, }, true); var sequences = this.options["sequences"]; - this.sequences_limit = sequences == 1 ? 2000 : sequences | 0; + this.sequences_limit = sequences == 1 ? 200 : sequences | 0; this.warnings_produced = {}; };