Merge pull request #933 from rollup/rollup-931

Only remove EmptyStatement nodes directly inside blocks.

src/ast/nodes/EmptyStatement.js

@@ -2,6 +2,8 @@ import Statement from './shared/Statement.js';
 
 export default class EmptyStatement extends Statement {
 	render ( code ) {
-		code.remove( this.start, this.end );
+		if ( this.parent.type === 'BlockStatement' || this.parent.type === 'Program' ) {
+			code.remove( this.start, this.end );
+		}
 	}
 }

test/form/empty-statement-consequent/_config.js

@@ -0,0 +1,3 @@
+module.exports = {
+	description: 'preserves empty statements used as the consequent of conditionals'
+};

test/form/empty-statement-consequent/_expected/amd.js

@@ -0,0 +1,6 @@
+define(function () { 'use strict';
+
+	if ( a === 0 );
+	else a++;
+
+});

test/form/empty-statement-consequent/_expected/cjs.js

@@ -0,0 +1,4 @@
+'use strict';
+
+if ( a === 0 );
+else a++;

test/form/empty-statement-consequent/_expected/es.js

@@ -0,0 +1,2 @@
+if ( a === 0 );
+else a++;

test/form/empty-statement-consequent/_expected/iife.js

@@ -0,0 +1,7 @@
+(function () {
+	'use strict';
+
+	if ( a === 0 );
+	else a++;
+
+}());

test/form/empty-statement-consequent/_expected/umd.js

@@ -0,0 +1,10 @@
+(function (global, factory) {
+	typeof exports === 'object' && typeof module !== 'undefined' ? factory() :
+	typeof define === 'function' && define.amd ? define(factory) :
+	(factory());
+}(this, (function () { 'use strict';
+
+	if ( a === 0 );
+	else a++;
+
+})));

test/form/empty-statement-consequent/main.js

@@ -0,0 +1,2 @@
+if ( a === 0 );
+else a++;