diff options
-rw-r--r-- | lib/compress.js | 17 | ||||
-rw-r--r-- | test/compress/spread.js | 30 |
2 files changed, 45 insertions, 2 deletions
diff --git a/lib/compress.js b/lib/compress.js index a78cf6ef..82474b5c 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -9689,13 +9689,26 @@ merge(Compressor.prototype, { }); }); + function is_integer(key) { + return /^[0-9]+$/.test(key); + } + OPT(AST_Spread, function(self, compressor) { if (compressor.option("properties")) { var exp = self.expression; if (compressor.parent() instanceof AST_Object) { if (exp instanceof AST_Object && all(exp.properties, function(node) { return node instanceof AST_ObjectKeyVal; - })) return List.splice(exp.properties); + })) return List.splice(exp.properties.map(function(node) { + var key = node.key; + if (!(key instanceof AST_Node) && is_integer(key)) { + node = node.clone(); + node.key = make_node(AST_Number, node, { + value: +key + }); + } + return node; + })); } else if (exp instanceof AST_Array) return List.splice(exp.elements.map(function(node) { return node instanceof AST_Hole ? make_node(AST_Undefined, node).optimize(compressor) : node; })); @@ -9971,7 +9984,7 @@ merge(Compressor.prototype, { var prop = self.properties[i]; var key = prop.key; if (key instanceof AST_Node) key = key.evaluate(compressor); - if (typeof key != "string" || /[0-9]+/.test(key)) break; + if (is_integer(key)) break; if (key !== prop.key) prop.key = "" + key; } var keys = new Dictionary(); diff --git a/test/compress/spread.js b/test/compress/spread.js index b7a6f753..01818dc3 100644 --- a/test/compress/spread.js +++ b/test/compress/spread.js @@ -369,3 +369,33 @@ unused_var_side_effects: { expect_stdout: "PASS" node_version: ">=8" } + +issue_4329: { + options = { + properties: true, + } + input: { + console.log({ + ...{ + get 0() { + return "FAIL"; + }, + ...{ + 0: "PASS", + }, + }, + }[0]); + } + expect: { + console.log({ + ...{ + get 0() { + return "FAIL"; + }, + [0]: "PASS", + }, + }[0]); + } + expect_stdout: "PASS" + node_version: ">=8" +} |