Make sure annotations are never handled as a node

rollup · Feb 26, 2019 · 41f01cb · 41f01cb
1 parent 115d76b
commit 41f01cb
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/ast/nodes/shared/Node.ts b/src/ast/nodes/shared/Node.ts
@@ -150,7 +150,7 @@ export class NodeBase implements ExpressionNode {
 	hasEffects(options: ExecutionPathOptions): boolean {
 		for (const key of this.keys) {
 			const value = (<GenericEsTreeNode>this)[key];
-			if (value === null) continue;
+			if (value === null || key === 'annotations') continue;
 			if (Array.isArray(value)) {
 				for (const child of value) {
 					if (child !== null && child.hasEffects(options)) return true;
@@ -180,7 +180,7 @@ export class NodeBase implements ExpressionNode {
 		this.included = true;
 		for (const key of this.keys) {
 			const value = (<GenericEsTreeNode>this)[key];
-			if (value === null) continue;
+			if (value === null || key === 'annotations') continue;
 			if (Array.isArray(value)) {
 				for (const child of value) {
 					if (child !== null) child.include(includeAllChildrenRecursively);
@@ -246,7 +246,7 @@ export class NodeBase implements ExpressionNode {
 	render(code: MagicString, options: RenderOptions) {
 		for (const key of this.keys) {
 			const value = (<GenericEsTreeNode>this)[key];
-			if (value === null) continue;
+			if (value === null || key === 'annotations') continue;
 			if (Array.isArray(value)) {
 				for (const child of value) {
 					if (child !== null) child.render(code, options);