Reverting "added option for dropping unused params"
authorMihai Bazon <mihai@bazon.net>
Thu, 25 Jul 2013 15:08:36 +0000 (18:08 +0300)
committerMihai Bazon <mihai@bazon.net>
Thu, 25 Jul 2013 15:08:36 +0000 (18:08 +0300)
Revert "added option for dropping unused params"

(turns out we already had the `unused` option for this.)

This reverts commit e54df2226f7f3887d2f850cea8caf5c0353dce00.

lib/compress.js

index c07ccd3..9eed828 100644 (file)
@@ -67,7 +67,6 @@ function Compressor(options, false_by_default) {
         cascade       : !false_by_default,
         side_effects  : !false_by_default,
         screw_ie8     : false,
-        drop_unused   : !false_by_default,
 
         warnings      : true,
         global_defs   : {}
@@ -84,7 +83,7 @@ merge(Compressor.prototype, {
     before: function(node, descend, in_list) {
         if (node._squeezed) return node;
         if (node instanceof AST_Scope) {
-            if (this.options.drop_unused) node.drop_unused(this);
+            node.drop_unused(this);
             node = node.hoist_declarations(this);
         }
         descend(node, this);
@@ -97,7 +96,7 @@ merge(Compressor.prototype, {
             // no point to repeat warnings.
             var save_warnings = this.options.warnings;
             this.options.warnings = false;
-            if (this.options.drop_unused) node.drop_unused(this);
+            node.drop_unused(this);
             this.options.warnings = save_warnings;
         }
         node._squeezed = true;