Extract function doesn't preserve semantic with return statements

[dbaeumer]

function foo(i: number) {
	let x = 10;
	for (let i = 0; i < 10; i++) {
		x++;
		return;
	}
}

• select

		x++;
		return;

• extract
• you get

function foo(i: number) {
	let x = 10;
	for (let i = 0; i < 10; i++) {
        x = newFunction(x);
	}
}
function newFunction(x: number) {
    x++;
    return x;
}

which is not sematic preserving.

[amcasey]

It only breaks when extracting into the global scope. I'm guessing it's failing to handle the combination of a write and a void return (e.g. it works if the original has return 1).