let + for-of does not respect scoping

[monsanto]

Minimal example:

let x = [0]
for (let x of x) {
    console.log(x)
}
console.log(x)

$ 6to5-node test.js
0
0

$ traceur test.js
0
[ 0 ]

[sebmck]

Odd. This should be working, I'll investigate shortly.

[sebmck]

This seems to transform correctly.

function foo() {
  var x = [0]
  for (let x of x) {
    console.log(x)
  }
  console.log(x)
}

foo();

Looks like a pretty serious bug I'll have to track down.

[monsanto]

@sebmck the example you just posted doesn't work for me, unless you are saying that you have just made fixes so it does

(I'm using 1.12.10)

[sebmck]

@monsanto I must have inadvertently fixed it kinda-ish when debugging. I'll have to write a lot more let scoping tests, Traceur's coverage doesn't seem to be enough.

[sebmck]

Looks like it's an issue with the scope tracking not tracking node position changes.

[sebmck]

Thanks! Fixed as of 1.12.12