Remove Empty Functions in block (close #247) (#248)

babel · Nov 3, 2016 · e8dbc90 · e8dbc90
1 parent 55ddb2b
commit e8dbc90
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 1 deletion.
diff --git a/packages/babel-plugin-minify-dead-code-elimination/__tests__/dead-code-elimination-test.js b/packages/babel-plugin-minify-dead-code-elimination/__tests__/dead-code-elimination-test.js
@@ -82,7 +82,7 @@ describe("dce-plugin", () => {
     const source = unpad(`
       function foo(p) {
         return 1;
-      };
+      }
       function bar(q) {
         return q + 1;
       }
@@ -2193,4 +2193,20 @@ describe("dce-plugin", () => {
       }
     })).toBe(expected);
   });
+  it("should remove empty statements when children of block", () => {
+    const source = unpad(`
+      (function () {
+        function foo() {};
+        function bar() {};
+        function baz() {};
+        function ban() {};
+        function quux() {};
+        function cake() {};
+      })();
+    `);
+    const expected = unpad(`
+      (function () {})();
+    `);
+    expect(transform(source)).toBe(expected);
+  });
 });
diff --git a/packages/babel-plugin-minify-dead-code-elimination/src/index.js b/packages/babel-plugin-minify-dead-code-elimination/src/index.js
@@ -678,6 +678,11 @@ module.exports = ({ types: t, traverse }) => {
   return {
     name: "minify-dead-code-elimination",
     visitor: {
+      EmptyStatement(path) {
+        if (path.parentPath.isBlockStatement() || path.parentPath.isProgram()) {
+          path.remove();
+        }
+      },
       Program(path, {
         opts: {
           // set defaults