Merge pull request #1712 from rollup/fix-1706

Fix issue when using labels inside functions

src/ast/ExecutionPathOptions.js

@@ -35,6 +35,16 @@ export default class ExecutionPathOptions {
 		return this._optionValues.get( option );
 	}
 
+	/**
+	 * Returns a new ExecutionPathOptions instance with the given option removed.
+	 * Does not mutate the current instance. Also works in sub-classes.
+	 * @param {string} option - The name of an option
+	 * @returns {*} Its value
+	 */
+	remove ( option ) {
+		return new this.constructor( this._optionValues.remove( option ) );
+	}
+
 	/**
 	 * Returns a new ExecutionPathOptions instance with the given option set to a new value.
 	 * Does not mutate the current instance. Also works in sub-classes.
@@ -245,7 +255,7 @@ export default class ExecutionPathOptions {
 	 * @return {ExecutionPathOptions}
 	 */
 	setIgnoreNoLabels () {
-		return this.set( OPTION_IGNORED_LABELS, null );
+		return this.remove( OPTION_IGNORED_LABELS );
 	}
 
 	/**

test/function/samples/handle-labels-inside-functions/_config.js

@@ -0,0 +1,3 @@
+module.exports = {
+	description: 'do not fail when using labels inside functions (#1706)'
+};

test/function/samples/handle-labels-inside-functions/main.js

@@ -0,0 +1,12 @@
+function loopWithLabel () {
+	label2: {
+		while ( true ) {
+			if ( Math.random() < 0.5 ) {
+				break label2;
+			}
+			console.log( 'loop' );
+		}
+	}
+}
+
+loopWithLabel();