From 8b890721903a0a983ffe74a3cecc6b922fa0518e Mon Sep 17 00:00:00 2001 From: kzc Date: Wed, 6 Sep 2017 14:44:26 -0400 Subject: [PATCH] add `Date` and other known globals to `unsafe` compress option (#2302) --- lib/compress.js | 2 +- test/compress/dead-code.js | 36 ++++++++++++++++++++++++++++++++++++ 2 files changed, 37 insertions(+), 1 deletion(-) diff --git a/lib/compress.js b/lib/compress.js index 6e766fb0..3da7a315 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -693,7 +693,7 @@ merge(Compressor.prototype, { return node instanceof AST_SymbolRef && node.definition().undeclared; } - var global_names = makePredicate("Array Boolean console Error Function Math Number RegExp Object String"); + var global_names = makePredicate("Array Boolean clearInterval clearTimeout console Date decodeURI decodeURIComponent encodeURI encodeURIComponent Error escape eval EvalError Function isFinite isNaN JSON Math Number parseFloat parseInt RangeError ReferenceError RegExp Object setInterval setTimeout String SyntaxError TypeError unescape URIError"); AST_SymbolRef.DEFMETHOD("is_declared", function(compressor) { return !this.definition().undeclared || compressor.option("unsafe") && global_names(this.name); diff --git a/test/compress/dead-code.js b/test/compress/dead-code.js index abf5297c..aea0e540 100644 --- a/test/compress/dead-code.js +++ b/test/compress/dead-code.js @@ -241,18 +241,54 @@ issue_2233_1: { Array.isArray; Boolean; console.log; + Date; + decodeURI; + decodeURIComponent; + encodeURI; + encodeURIComponent; Error.name; + escape; + eval; + EvalError; Function.length; + isFinite; + isNaN; + JSON; Math.random; Number.isNaN; + parseFloat; + parseInt; RegExp; Object.defineProperty; String.fromCharCode; + RangeError; + ReferenceError; + SyntaxError; + TypeError; + unescape; + URIError; } expect: {} expect_stdout: true } +global_timeout_and_interval_symbols: { + options = { + pure_getters: "strict", + side_effects: true, + unsafe: true, + } + input: { + // These global symbols do not exist in the test sandbox + // and must be tested separately. + clearInterval; + clearTimeout; + setInterval; + setTimeout; + } + expect: {} +} + issue_2233_2: { options = { pure_getters: "strict", -- 2.34.1