KarpelesLab / goro / 25772047510

13%

Build Type

push

github

Committed by MagicalTux

Commit Message

vm: clone, instanceof, do-while, declare, var-var, generator iter

Six more coverage extensions, all via AST delegation:

* \`clone \$x\`, \`\$x instanceof T\`, void cast (used implicitly),
  first-class callables — push result via OpClassConst.
* \`do { … } while (…);\`, \`declare(strict_types=1)\`, \`declare(ticks=N)\`,
  top-level const, enum register, inline HTML — statement form via
  OpTryFinally + OP_REFRESH_SLOTS.
* Dynamic object property name (\`\$obj->{\$expr}\`) and dynamic
  method name (\`\$obj->{\$expr}()\`).

VM foreach over an object implementing the Iterator interface
(notably Generator) now drives the iteration via PHP-level
rewind/valid/current/key/next methods through the AST's
phpObjectIterator (re-exposed as compiler.NewObjectMethodIterator).
Previously the VM used NewIteratorInScope, which iterates over
object properties — wrong for generators and Iterator classes.

Tests: TestCloneInstanceofDoWhile, TestGeneratorIteration. The
unsupported-fallback sentinel is retired since the emitter no
longer has a reachable catch-all for valid scripts; fallback for
generators / by-ref params / by-ref return is still covered by
the integration test.

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

Coverage Stats

41 of 44 new or added lines in 5 files covered. (93.18%)

21 existing lines in 6 files now uncovered.

13859 of 115356 relevant lines covered (12.01%)

2017.59 hits per line