Minified/Uglified Issue

[chriscoyier]

Say you take the entire https://cdnjs.cloudflare.com/ajax/libs/react/15.4.2/react.min.js and instrument it with what is here.

You'll get some invalid JS. Here's a formatted-ish chunk of the broken JS:

if (window.CP.shouldStopExecution(22)){break;}
  d=g.value,y=h+r(d,x++),v+=o(d,y,n,i);}

// Invalid right here.
window.CP.exitedLoop(22);

else for(;!(g=E.next()).done;){if (window.CP.shouldStopExecution(23)){break;}var _=g.value;_&&(d=_[1],y=h+c.escape(_[0])+f+r(d,0),v+=o(d,y,n,i))}
window.CP.exitedLoop(23);
}

I added the stuff required for testing as 8-in.js and 8-out-broken.js.

You can run node output-checker.js to instrument and see results quickly.

[ariya]

A reduced test case is the following:

if (1)
    for (;;) { doSomething(); }
else {
    doThat();
}

that got modified (incorrectly) to:

if (1)
    for (;;) {if (window.CP.shouldStopExecution(1)){break;} doSomething(); }
window.CP.exitedLoop(1);

else {
    doThat();
}

which is of course invalid because that else is dangling (since the for loop is a clause inside the if statement, and it is not enclosed by curly braces to form a block).

I believe the fix is to ensure that a loop is enclosed properly. Let me think of something.

[normlee1977]

• #d=g.value,y=h+r(d,x++),v+=o(d,y,
  Duplicate of Loop Case #2

[InfiniteLoop2668619f04c1820a204676c8aa187709d87cd.zip](https://github.com/codepen/InfiniteLoopBuster/files/6824830