Do not apply negate_iife optimization to `new` expression
authorkzc <zaxxon2011@gmail.com>
Mon, 9 May 2016 07:19:28 +0000 (03:19 -0400)
committerkzc <zaxxon2011@gmail.com>
Mon, 9 May 2016 07:19:28 +0000 (03:19 -0400)
lib/compress.js
test/compress/negate-iife.js

index 2bcfcf3..1d6a943 100644 (file)
@@ -811,6 +811,11 @@ merge(Compressor.prototype, {
                 if (stat instanceof AST_SimpleStatement) {
                     stat.body = (function transform(thing) {
                         return thing.transform(new TreeTransformer(function(node){
+                            if (node instanceof AST_New
+                                && node.expression instanceof AST_Call
+                                && node.expression.expression instanceof AST_Function) {
+                                return node;
+                            }
                             if (node instanceof AST_Call && node.expression instanceof AST_Function) {
                                 return make_node(AST_UnaryPrefix, node, {
                                     operator: "!",
index 89c3f06..20b3f56 100644 (file)
@@ -74,3 +74,27 @@ negate_iife_4: {
         }();
     }
 }
+
+negate_iife_issue_1073: {
+    options = {
+        negate_iife: true,
+        sequences: true,
+        conditionals: true,
+    };
+    input: {
+        new (function(a) {
+            return function Foo() {
+                this.x = a;
+                console.log(this);
+            };
+        }(7))();
+    }
+    expect: {
+        new (function(a) {
+            return function Foo() {
+                this.x = a,
+                console.log(this);
+            };
+        }(7))();
+    }
+}