ModuleInstrumenter passing null in an (possibly?) inapporpriate manner
peteryongzhong opened this issue · 0 comments
peteryongzhong commented
In here:
I feel like for here passing null will cause the following to fail the null vibe check.
wizard-engine/src/util/ProbeUtil.v3
Line 36 in 12c03ba
Perhaps the strat would be, in the absence of a lambda implementation in virgil, to add some noop function and use that as the argument? But perhaps i am missing some bigger picture.
def noop(d: DynamicLoc) -> Resumption{
return Resumption.Continue;
}