dxgriffiths/JS11 — Issue #8: Implied for loop variables may not be unique
Author: posniewski
Created: 2011-12-01 00:39:20+00:00
State: open

I noticed that Math.Random is used to create a unique-ish identifier for the lop variable. Technically, these numbers may repeat (though the chances are pretty small). In my fork I've changed this to use the character index of match for the loop. This should be unique with the file (as each replacement is strictly larger, subsequent matches will always have larger indexes.

What do you think?