JoBoEVD/UglifyJS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

renamed --screw-ie to --screw-ie8, documented it in README.md, indicated it doesn't break IE9+

Browse Source
This commit is contained in:
Michał Gołębiowski
2013-03-25 16:11:07 +01:00
parent 478bf4dbdd
commit 75f0bbe6e8
3 changed files with 11 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
README.md
View File

@@ -54,6 +54,9 @@ The available options are:
--source-map. [string] --source-map. [string]
--in-source-map Input source map, useful if you're compressing JS that was --in-source-map Input source map, useful if you're compressing JS that was
generated from some other original code. generated from some other original code.
--screw-ie8 Pass this flag if you don't care about full compliance with
Internet Explorer 6-8 quirks (by default UglifyJS will try
to be IE-proof.
-p, --prefix Skip prefix for original filenames that appear in source -p, --prefix Skip prefix for original filenames that appear in source
maps. For example -p 3 will drop 3 directories from file maps. For example -p 3 will drop 3 directories from file
names and ensure they are relative paths. names and ensure they are relative paths.

12
bin/uglifyjs
View File

@@ -21,7 +21,7 @@ mangling you need to use `-c` and `-m`.\
.describe("source-map-root", "The path to the original source to be included in the source map.") .describe("source-map-root", "The path to the original source to be included in the source map.")
.describe("source-map-url", "The path to the source map to be added in //@ sourceMappingURL. Defaults to the value passed with --source-map.") .describe("source-map-url", "The path to the source map to be added in //@ sourceMappingURL. Defaults to the value passed with --source-map.")
.describe("in-source-map", "Input source map, useful if you're compressing JS that was generated from some other original code.") .describe("in-source-map", "Input source map, useful if you're compressing JS that was generated from some other original code.")
.describe("screw-ie", "Pass this flag if you don't care about full compliance with Internet Explorer quirks (by default UglifyJS will try to be IE-proof).") .describe("screw-ie8", "Pass this flag if you don't care about full compliance with Internet Explorer 6-8 quirks (by default UglifyJS will try to be IE-proof).")
.describe("p", "Skip prefix for original filenames that appear in source maps. \ .describe("p", "Skip prefix for original filenames that appear in source maps. \
For example -p 3 will drop 3 directories from file names and ensure they are relative paths.") For example -p 3 will drop 3 directories from file names and ensure they are relative paths.")
.describe("o", "Output file (default STDOUT).") .describe("o", "Output file (default STDOUT).")
@@ -75,7 +75,7 @@ You need to pass an argument to this option to specify the name that your module
.string("e") .string("e")
.string("comments") .string("comments")
.string("wrap") .string("wrap")
.boolean("screw-ie") .boolean("screw-ie8")
.boolean("export-all") .boolean("export-all")
.boolean("self") .boolean("self")
.boolean("v") .boolean("v")
@@ -125,8 +125,8 @@ if (MANGLE && ARGS.r) {
MANGLE.except = ARGS.r.replace(/^\s+|\s+$/g).split(/\s*,+\s*/); MANGLE.except = ARGS.r.replace(/^\s+|\s+$/g).split(/\s*,+\s*/);
} }
if (MANGLE && ARGS.screw_ie) { if (MANGLE && ARGS.screw_ie8) {
MANGLE.screw_ie = true; MANGLE.screw_ie8 = true;
} }
var OUTPUT_OPTIONS = { var OUTPUT_OPTIONS = {
@@ -265,7 +265,7 @@ var SCOPE_IS_NEEDED = COMPRESS || MANGLE || ARGS.lint;
if (SCOPE_IS_NEEDED) { if (SCOPE_IS_NEEDED) {
time_it("scope", function(){ time_it("scope", function(){
TOPLEVEL.figure_out_scope({ screw_ie: ARGS.screw_ie }); TOPLEVEL.figure_out_scope({ screw_ie8: ARGS.screw_ie8 });
if (ARGS.lint) { if (ARGS.lint) {
TOPLEVEL.scope_warnings(); TOPLEVEL.scope_warnings();
} }
@@ -280,7 +280,7 @@ if (COMPRESS) {
if (SCOPE_IS_NEEDED) { if (SCOPE_IS_NEEDED) {
time_it("scope", function(){ time_it("scope", function(){
TOPLEVEL.figure_out_scope({ screw_ie: ARGS.screw_ie }); TOPLEVEL.figure_out_scope({ screw_ie8: ARGS.screw_ie8 });
if (MANGLE) { if (MANGLE) {
TOPLEVEL.compute_char_frequency(MANGLE); TOPLEVEL.compute_char_frequency(MANGLE);
} }

4
lib/scope.js
View File

@@ -64,7 +64,7 @@ SymbolDef.prototype = {
mangle: function(options) { mangle: function(options) {
if (!this.mangled_name && !this.unmangleable(options)) { if (!this.mangled_name && !this.unmangleable(options)) {
var s = this.scope; var s = this.scope;
if (this.orig[0] instanceof AST_SymbolLambda && !options.screw_ie) if (this.orig[0] instanceof AST_SymbolLambda && !options.screw_ie8)
s = s.parent_scope; s = s.parent_scope;
this.mangled_name = s.next_mangled(options); this.mangled_name = s.next_mangled(options);
} }
@@ -343,7 +343,7 @@ AST_Toplevel.DEFMETHOD("_default_mangler_options", function(options){
eval : false, eval : false,
sort : false, sort : false,
toplevel : false, toplevel : false,
screw_ie : false screw_ie8 : false
}); });
}); });