allow empty for loop heads – fixes #919

src/ast/nodes/ForStatement.js

@@ -1,15 +1,6 @@
 import Statement from './shared/Statement.js';
 
 export default class ForStatement extends Statement {
-	bind () {
-		const scope = this.body.scope;
-
-		this.init.bind( scope );
-		this.test.bind( scope );
-		this.update.bind( scope );
-		this.body.bind( scope );
-	}
-
 	hasEffects () {
 		return super.hasEffects( this.body.scope );
 	}
@@ -19,9 +10,9 @@ export default class ForStatement extends Statement {
 		scope = this.body.scope;
 
 		// can't use super, because we need to control the order
-		this.init.initialise( scope );
-		this.test.initialise( scope );
-		this.update.initialise( scope );
+		if ( this.init ) this.init.initialise( scope );
+		if ( this.test ) this.test.initialise( scope );
+		if ( this.update ) this.update.initialise( scope );
 		this.body.initialise( scope );
 	}
 

test/form/for-loop-with-empty-head/_config.js

@@ -0,0 +1,3 @@
+module.exports = {
+	description: 'handles for loop with empty head'
+};

test/form/for-loop-with-empty-head/_expected/amd.js

@@ -0,0 +1,7 @@
+define(function () { 'use strict';
+
+	for ( ; ; ) {
+		console.log( 42 );
+	}
+
+});

test/form/for-loop-with-empty-head/_expected/cjs.js

@@ -0,0 +1,5 @@
+'use strict';
+
+for ( ; ; ) {
+	console.log( 42 );
+}

test/form/for-loop-with-empty-head/_expected/es.js

@@ -0,0 +1,3 @@
+for ( ; ; ) {
+	console.log( 42 );
+}

test/form/for-loop-with-empty-head/_expected/iife.js

@@ -0,0 +1,8 @@
+(function () {
+	'use strict';
+
+	for ( ; ; ) {
+		console.log( 42 );
+	}
+
+}());

test/form/for-loop-with-empty-head/_expected/umd.js

@@ -0,0 +1,11 @@
+(function (global, factory) {
+	typeof exports === 'object' && typeof module !== 'undefined' ? factory() :
+	typeof define === 'function' && define.amd ? define(factory) :
+	(factory());
+}(this, (function () { 'use strict';
+
+	for ( ; ; ) {
+		console.log( 42 );
+	}
+
+})));

test/form/for-loop-with-empty-head/main.js

@@ -0,0 +1,3 @@
+for ( ; ; ) {
+	console.log( 42 );
+}