fix reduce_vars on AST_Array.length (#2404)
This commit is contained in:
@@ -572,7 +572,9 @@ merge(Compressor.prototype, {
|
|||||||
if (key instanceof AST_Node) return null;
|
if (key instanceof AST_Node) return null;
|
||||||
var value;
|
var value;
|
||||||
if (obj instanceof AST_Array) {
|
if (obj instanceof AST_Array) {
|
||||||
value = obj.elements[key];
|
var elements = obj.elements;
|
||||||
|
if (key == "length") return make_node_from_constant(elements.length, obj);
|
||||||
|
if (typeof key == "number" && key in elements) value = elements[key];
|
||||||
} else if (obj instanceof AST_Object) {
|
} else if (obj instanceof AST_Object) {
|
||||||
var props = obj.properties;
|
var props = obj.properties;
|
||||||
for (var i = props.length; --i >= 0;) {
|
for (var i = props.length; --i >= 0;) {
|
||||||
|
|||||||
@@ -131,6 +131,7 @@ for_loop: {
|
|||||||
evaluate: true,
|
evaluate: true,
|
||||||
reduce_vars: true,
|
reduce_vars: true,
|
||||||
unsafe: true,
|
unsafe: true,
|
||||||
|
unused: true,
|
||||||
};
|
};
|
||||||
input: {
|
input: {
|
||||||
function f0() {
|
function f0() {
|
||||||
@@ -166,7 +167,7 @@ for_loop: {
|
|||||||
function f1() {
|
function f1() {
|
||||||
var a = [1, 2, 3];
|
var a = [1, 2, 3];
|
||||||
var b = 0;
|
var b = 0;
|
||||||
for (var i = 0, len = a.length; i < len; i++)
|
for (var i = 0; i < 3; i++)
|
||||||
b += a[i];
|
b += a[i];
|
||||||
return b;
|
return b;
|
||||||
}
|
}
|
||||||
@@ -180,3 +181,57 @@ for_loop: {
|
|||||||
}
|
}
|
||||||
expect_stdout: "6 6 4"
|
expect_stdout: "6 6 4"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
index: {
|
||||||
|
options = {
|
||||||
|
evaluate: true,
|
||||||
|
reduce_vars: true,
|
||||||
|
toplevel: true,
|
||||||
|
unsafe: true,
|
||||||
|
unused: true,
|
||||||
|
}
|
||||||
|
input: {
|
||||||
|
var a = [ 1, 2 ];
|
||||||
|
console.log(a[0], a[1]);
|
||||||
|
}
|
||||||
|
expect: {
|
||||||
|
console.log(1, 2);
|
||||||
|
}
|
||||||
|
expect_stdout: "1 2"
|
||||||
|
}
|
||||||
|
|
||||||
|
length: {
|
||||||
|
options = {
|
||||||
|
evaluate: true,
|
||||||
|
reduce_vars: true,
|
||||||
|
toplevel: true,
|
||||||
|
unsafe: true,
|
||||||
|
unused: true,
|
||||||
|
}
|
||||||
|
input: {
|
||||||
|
var a = [ 1, 2 ];
|
||||||
|
console.log(a.length);
|
||||||
|
}
|
||||||
|
expect: {
|
||||||
|
console.log(2);
|
||||||
|
}
|
||||||
|
expect_stdout: "2"
|
||||||
|
}
|
||||||
|
|
||||||
|
index_length: {
|
||||||
|
options = {
|
||||||
|
evaluate: true,
|
||||||
|
reduce_vars: true,
|
||||||
|
toplevel: true,
|
||||||
|
unsafe: true,
|
||||||
|
unused: true,
|
||||||
|
}
|
||||||
|
input: {
|
||||||
|
var a = [ 1, 2 ];
|
||||||
|
console.log(a[0], a.length);
|
||||||
|
}
|
||||||
|
expect: {
|
||||||
|
console.log(1, 2);
|
||||||
|
}
|
||||||
|
expect_stdout: "1 2"
|
||||||
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user