KarpelesLab / goro / 25814669282

13%

push

github

vm: unwind foreach iterators on multi-level break/continue

\`continue 2\` from inside a nested foreach was jumping straight to
the outer loop's continue target without popping the inner
foreach's iterator off the iters stack. The next OpForeachStep
then read the wrong (still-inner) iterator, looping forever.

Tag loopCtx with isForeach when the loop is a foreach, and at
break/continue emit OP_FOREACH_UNWIND for every foreach in the
loops being skipped over.

Fixes bug67111 ("Memory leak when using 'continue 2' inside two
foreach loops" — used here as a correctness test) and any other
multi-level break/continue across foreach.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

7 of 9 new or added lines in 1 file covered. (77.78%)

13971 of 115654 relevant lines covered (12.08%)

2454.66 hits per line