JimmyZ docs.libuv.org/en/v1.x/#documentation # better docs for libuv, FYI 12:21
dalek arVM: f3512a8 | paultcochrane++ | src/6model/reprs/MVMCode.c:
Free allocated variable before returning

Coverity scan pointed out that the variable `ann` wasn't being freed before the function MVM_code_location() returned. This should fix this issue. The NQP tests all pass after having made this change (there aren't any MoarVM tests), so it seems that everything still works as expected.		 17:07
arVM: 522e168 | timo++ | src/6model/reprs/MVMCode.c:
Merge pull request #203 from paultcochrane/pr/fix_MVMCode_resource_leak

Free allocated variable before returning 88592c7 | jnthn++ | src/ (3 files): More missing frees of bytecode annotations.
timotimo [ptc]++
jnthn f3512a8 looks legit, but I now think we should check other callsites for that function :) 19:23
timotimo yeah
i can do that
jnthn already found 2 here :) 19:26
(and written le patches) 19:27
timotimo i see 19:28
timotimo at least the one in the profiler and spesh seem "hot" 19:30
so that's very good 19:31
though since the data is apparently always extremely short-lived, we may want to allocate it on the stack of the caller and populate it in the callee
jnthn maybe, though none of it is hot path, afaik
timotimo yeah 19:32
just a little tingling sensation in my nose :P
:-P

Powered by Cro and the Raku® Programming Language.

Please report any issues / comments / feature requests as an issue on App::Raku::Log.

Thank you!