KarpelesLab / goro / 26359766848

12%

push

github

vm: drop now-dead dollar-prefix fallback in emitObjectVarRead/Func + remove unused emitCallViaAST

$obj->{$x} parses to runObjectDynVar (intercepted by IsObjectDynVarReadNode)
and $obj->{$x}() parses to runObjectDynFunc (intercepted by IsObjectDynFuncNode),
both before objectVarNode/objectFuncNode dispatch is reached. The name[0]=='$'
branches in emitObjectVarRead / emitObjectFuncCall were therefore unreachable.

Removing them also makes emitCallViaAST unused — drop it entirely.

No behavioral change. Smoke tests for dyn-name read / dyn-name call /
literal read / literal call all pass.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

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

14755 of 118205 relevant lines covered (12.48%)

2586.29 hits per line