.describe("wrap", "Embed everything in a big function, making the “exports” and “global” variables available. \
You need to pass an argument to this option to specify the name that your module will take when included in, say, a browser.")
.describe("export-all", "Only used when --wrap, this tells UglifyJS to add code to automatically export all globals.")
+ .describe("lint", "Display some scope warnings")
.describe("v", "Verbose")
.alias("p", "prefix")
.boolean("stats")
.boolean("acorn")
.boolean("spidermonkey")
+ .boolean("lint")
.wrap(80)
TOPLEVEL = TOPLEVEL.wrap_commonjs(ARGS.wrap, ARGS.export_all);
}
-var SCOPE_IS_NEEDED = COMPRESS || MANGLE;
+var SCOPE_IS_NEEDED = COMPRESS || MANGLE || ARGS.lint;
if (SCOPE_IS_NEEDED) {
time_it("scope", function(){
TOPLEVEL.figure_out_scope();
+ if (ARGS.lint) {
+ TOPLEVEL.scope_warnings();
+ }
});
}
// XXX: this also warns about JS standard names,
// i.e. Object, Array, parseInt etc. Should add a list of
// exceptions.
- AST_Node.warn("Undeclared symbol: {name} [{line},{col}]", {
+ AST_Node.warn("Undeclared symbol: {name} [{file}:{line},{col}]", {
name: node.name,
+ file: node.start.file,
line: node.start.line,
col: node.start.col
});
if (sym
&& (sym.undeclared()
|| (sym.global() && sym.scope !== sym.definition().scope))) {
- AST_Node.warn("{msg}: {name} [{line},{col}]", {
+ AST_Node.warn("{msg}: {name} [{file}:{line},{col}]", {
msg: sym.undeclared() ? "Accidental global?" : "Assignment to global",
name: sym.name,
+ file: sym.start.file,
line: sym.start.line,
col: sym.start.col
});
&& node instanceof AST_SymbolRef
&& node.undeclared()
&& node.name == "eval") {
- AST_Node.warn("Eval is used [{line},{col}]", node.start);
+ AST_Node.warn("Eval is used [{file}:{line},{col}]", node.start);
}
if (options.unreferenced
&& node instanceof AST_SymbolDeclaration
&& node.unreferenced()) {
- AST_Node.warn("{type} {name} is declared but not referenced [{line},{col}]", {
+ AST_Node.warn("{type} {name} is declared but not referenced [{file}:{line},{col}]", {
type: node instanceof AST_Label ? "Label" : "Symbol",
name: node.name,
+ file: node.start.file,
line: node.start.line,
col: node.start.col
});
if (options.func_arguments
&& node instanceof AST_Lambda
&& node.uses_arguments) {
- AST_Node.warn("arguments used in function {name} [{line},{col}]", {
+ AST_Node.warn("arguments used in function {name} [{file}:{line},{col}]", {
name: node.name ? node.name.name : "anonymous",
+ file: node.start.file,
line: node.start.line,
col: node.start.col
});
if (options.nested_defuns
&& node instanceof AST_Defun
&& !(tw.parent() instanceof AST_Scope)) {
- AST_Node.warn("Function {name} declared in nested statement [{line},{col}]", {
+ AST_Node.warn("Function {name} declared in nested statement \"{type}\" [{file}:{line},{col}]", {
name: node.name.name,
+ type: tw.parent().TYPE,
+ file: node.start.file,
line: node.start.line,
col: node.start.col
});