Make sure Program nodes are really only included if they have an effect

rollup · Apr 16, 2018 · 3e1c6fc · 3e1c6fc
1 parent 93f75b6
commit 3e1c6fc
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/Module.ts b/src/Module.ts
@@ -562,7 +562,7 @@ export default class Module {
 	}
 
 	include() {
-		return this.ast.include();
+		return this.ast.shouldBeIncluded() && this.ast.include();
 	}
 
 	namespace(): NamespaceVariable {

diff --git a/src/ast/nodes/Program.ts b/src/ast/nodes/Program.ts
@@ -2,12 +2,19 @@ import MagicString from 'magic-string';
 import { NodeBase, StatementNode } from './shared/Node';
 import { NodeType } from './NodeType';
 import { RenderOptions, renderStatementList } from '../../utils/renderHelpers';
+import ExecutionPathOptions from '../ExecutionPathOptions';
 
 export default class Program extends NodeBase {
 	type: NodeType.Program;
 	body: StatementNode[];
 	sourceType: 'module';
 
+	hasEffects(options: ExecutionPathOptions) {
+		for (const node of this.body) {
+			if (node.hasEffects(options)) return true;
+		}
+	}
+
 	include() {
 		let anotherPassNeeded = false;
 		this.included = true;