From 102d1b9137353086f11fe2fc7abf9095042d4306 Mon Sep 17 00:00:00 2001 From: kzc Date: Sat, 27 Feb 2016 15:33:10 -0500 Subject: [PATCH] #877 Ignore mangle sort option --- README.md | 5 ----- lib/scope.js | 5 +---- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/README.md b/README.md index 4012b755bda..92559019282 100644 --- a/README.md +++ b/README.md @@ -192,11 +192,6 @@ input files from the command line. To enable the mangler you need to pass `--mangle` (`-m`). The following (comma-separated) options are supported: -- `sort` — to assign shorter names to most frequently used variables. This - saves a few hundred bytes on jQuery before gzip, but the output is - _bigger_ after gzip (and seems to happen for other libraries I tried it - on) therefore it's not enabled by default. - - `toplevel` — mangle names declared in the toplevel scope (disabled by default). diff --git a/lib/scope.js b/lib/scope.js index ace25894d65..5ab775a34b1 100644 --- a/lib/scope.js +++ b/lib/scope.js @@ -372,7 +372,7 @@ AST_Toplevel.DEFMETHOD("_default_mangler_options", function(options){ return defaults(options, { except : [], eval : false, - sort : false, + sort : false, // Ignored. Flag retained for backwards compatibility. toplevel : false, screw_ie8 : false, keep_fnames : false @@ -415,9 +415,6 @@ AST_Toplevel.DEFMETHOD("mangle_names", function(options){ a.push(symbol); } }); - if (options.sort) a.sort(function(a, b){ - return b.references.length - a.references.length; - }); to_mangle.push.apply(to_mangle, a); return; }