From a0c3836ba0e3baca8d758d9855144ac15ec0ac8d Mon Sep 17 00:00:00 2001 From: "Alex Lam S.L" Date: Fri, 31 Mar 2017 16:41:04 +0800 Subject: [PATCH] sort options in alphabetical order (#1743) They started off as functional groups I guess, but given the sheer number of options this is becoming too difficult to read. --- lib/compress.js | 50 +++++++++++++++++++++++------------------------ lib/output.js | 28 +++++++++++++------------- lib/parse.js | 10 +++++----- lib/propmangle.js | 12 ++++++------ lib/scope.js | 14 ++++++------- tools/node.js | 20 +++++++++---------- 6 files changed, 67 insertions(+), 67 deletions(-) diff --git a/lib/compress.js b/lib/compress.js index ac7ea357..55f6d793 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -48,42 +48,42 @@ function Compressor(options, false_by_default) { return new Compressor(options, false_by_default); TreeTransformer.call(this, this.before, this.after); this.options = defaults(options, { - sequences : !false_by_default, - properties : !false_by_default, + angular : false, + booleans : !false_by_default, + cascade : !false_by_default, + collapse_vars : !false_by_default, + comparisons : !false_by_default, + conditionals : !false_by_default, dead_code : !false_by_default, + drop_console : false, drop_debugger : !false_by_default, - unsafe : false, - unsafe_comps : false, - unsafe_math : false, - unsafe_proto : false, - conditionals : !false_by_default, - comparisons : !false_by_default, evaluate : !false_by_default, - booleans : !false_by_default, - loops : !false_by_default, - unused : !false_by_default, - toplevel : !!(options && options["top_retain"]), - top_retain : null, + expression : false, + global_defs : {}, hoist_funs : !false_by_default, - keep_fargs : true, - keep_fnames : false, hoist_vars : false, if_return : !false_by_default, join_vars : !false_by_default, - collapse_vars : !false_by_default, - reduce_vars : !false_by_default, - cascade : !false_by_default, - side_effects : !false_by_default, + keep_fargs : true, + keep_fnames : false, + loops : !false_by_default, + negate_iife : !false_by_default, + passes : 1, + properties : !false_by_default, pure_getters : false, pure_funcs : null, - negate_iife : !false_by_default, + reduce_vars : !false_by_default, screw_ie8 : true, - drop_console : false, - angular : false, - expression : false, + sequences : !false_by_default, + side_effects : !false_by_default, + top_retain : null, + toplevel : !!(options && options["top_retain"]), + unsafe : false, + unsafe_comps : false, + unsafe_math : false, + unsafe_proto : false, + unused : !false_by_default, warnings : true, - global_defs : {}, - passes : 1, }, true); var pure_funcs = this.options["pure_funcs"]; if (typeof pure_funcs == "function") { diff --git a/lib/output.js b/lib/output.js index ac9e0654..d5c7304a 100644 --- a/lib/output.js +++ b/lib/output.js @@ -53,26 +53,26 @@ function is_some_comments(comment) { function OutputStream(options) { options = defaults(options, { - indent_start : 0, - indent_level : 4, - quote_keys : false, - space_colon : true, ascii_only : false, - unescape_regexps : false, - inline_script : false, - width : 80, - max_line_len : false, beautify : false, - source_map : null, bracketize : false, - semicolons : true, comments : false, - shebang : true, - preserve_line : false, - screw_ie8 : true, + indent_level : 4, + indent_start : 0, + inline_script : false, + keep_quoted_props: false, + max_line_len : false, preamble : null, + preserve_line : false, + quote_keys : false, quote_style : 0, - keep_quoted_props: false, + screw_ie8 : true, + semicolons : true, + shebang : true, + source_map : null, + space_colon : true, + unescape_regexps : false, + width : 80, wrap_iife : false, }, true); diff --git a/lib/parse.js b/lib/parse.js index 1ccde26a..c34e13db 100644 --- a/lib/parse.js +++ b/lib/parse.js @@ -688,14 +688,14 @@ var ATOMIC_START_TOKEN = array_to_hash([ "atom", "num", "string", "regexp", "nam function parse($TEXT, options) { options = defaults(options, { - strict : false, - filename : null, - toplevel : null, + bare_returns : false, + cli : false, expression : false, + filename : null, html5_comments : true, - bare_returns : false, shebang : true, - cli : false, + strict : false, + toplevel : null, }); var S = { diff --git a/lib/propmangle.js b/lib/propmangle.js index 3c75cac9..cfa035d2 100644 --- a/lib/propmangle.js +++ b/lib/propmangle.js @@ -62,12 +62,12 @@ function find_builtins() { function mangle_properties(ast, options) { options = defaults(options, { - reserved : null, - cache : null, - only_cache : false, - regex : null, - ignore_quoted : false, - debug : false + cache: null, + debug: false, + ignore_quoted: false, + only_cache: false, + regex: null, + reserved: null, }); var reserved = options.reserved; diff --git a/lib/scope.js b/lib/scope.js index df0a7e00..74760e4f 100644 --- a/lib/scope.js +++ b/lib/scope.js @@ -92,8 +92,8 @@ SymbolDef.prototype = { AST_Toplevel.DEFMETHOD("figure_out_scope", function(options){ options = defaults(options, { + cache: null, screw_ie8: true, - cache: null }); // pass 1: setup scope chaining and handle definitions @@ -398,12 +398,12 @@ AST_Symbol.DEFMETHOD("global", function(){ AST_Toplevel.DEFMETHOD("_default_mangler_options", function(options){ return defaults(options, { - except : [], eval : false, + except : [], + keep_fnames : false, + screw_ie8 : true, sort : false, // Ignored. Flag retained for backwards compatibility. toplevel : false, - screw_ie8 : true, - keep_fnames : false }); }); @@ -576,12 +576,12 @@ var base54 = (function() { AST_Toplevel.DEFMETHOD("scope_warnings", function(options){ options = defaults(options, { - undeclared : false, // this makes a lot of noise - unreferenced : true, assign_to_global : true, + eval : true, func_arguments : true, nested_defuns : true, - eval : true + undeclared : false, // this makes a lot of noise + unreferenced : true, }); var tw = new TreeWalker(function(node){ if (options.undeclared diff --git a/tools/node.js b/tools/node.js index 6568a741..147751ae 100644 --- a/tools/node.js +++ b/tools/node.js @@ -46,21 +46,21 @@ function read_source_map(code) { UglifyJS.minify = function(files, options) { options = UglifyJS.defaults(options, { - spidermonkey : false, - outSourceMap : null, - outFileName : null, - sourceRoot : null, - inSourceMap : null, - sourceMapUrl : null, - sourceMapInline : false, + compress : {}, fromString : false, - warnings : false, + inSourceMap : null, mangle : {}, mangleProperties : false, nameCache : null, + outFileName : null, output : null, - compress : {}, - parse : {} + outSourceMap : null, + parse : {}, + sourceMapInline : false, + sourceMapUrl : null, + sourceRoot : null, + spidermonkey : false, + warnings : false, }); UglifyJS.base54.reset(); -- 2.34.1