KarpelesLab / goro / 25772972887

13%

Build Type

push

github

Committed by MagicalTux

Commit Message

vm: conservative by-ref handling for unknown callees and indirect calls

When emitFunctionCall hits a function name that isn't yet visible in
the lazy-function table (compiled inside an outer scope before the
top-level RegisterLazyFunc pre-pass) we can't know its signature.
Combined with a writable lvalue arg, that gap silently breaks by-ref
binding. Likewise for indirect (\$expr(...)) calls — the callable is
resolved at runtime, so the compile-time signature is unknowable.

Both routes now AST-delegate when the unknown callable receives any
writable lvalue. IsSlotSafe mirrors the same gating so the hashtable
stays populated for the AST run.

Adds LookupLazyByRefSafe as a nil-tolerant wrapper.

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

Coverage Stats

24 of 27 new or added lines in 2 files covered. (88.89%)

8 existing lines in 1 file now uncovered.

14014 of 115511 relevant lines covered (12.13%)

2041.75 hits per line