Fork me on GitHub
#core-async
<
2022-07-08
>
Alex Miller (Clojure team)18:07:09

@hiredman is this also true of your patches?

Alex Miller (Clojure team)18:07:23

should I look at them, or not look at them?

hiredman18:07:12

oh, look at them, the first patch in the series changes the rawcode stuff to only reconstruct the environment that the expression actually needs

hiredman18:07:08

to be the clear, the changes in those patches seem to have removed the oome from the example on the ticket, and I don't get the oome from the one other linked ticket I tried either

hiredman18:07:37

in testing it was a little tricky, since triggering the oome with a finite range size seemed hit or miss

hiredman18:07:19

inspecting the macro expansion of the into-chan definition with the patches applied, it doesn't store coll in the state vector any more, and ch and close? get nil'ed after the last use (but that doesn't effect the oome and is as the loop is exiting)