ok, good to know

@dnolen Any chance to get this merged? https://dev.clojure.org/jira/browse/CLJS-1965 (letfn is broken). We just ran into this and it caused a lot of head-scratching. It doesn't seem like "Minor" is the correct priority...

well nm, it needs tests

@mfikes is it possible to use your coal-mine apparatus on a local clojurescript build with a patch applied?

@favila marked as minor as it’s top-level letfn not something I would necessarily consider idiomatic

but yes, need some tests for that patch, happy to take as a separate patch

updated ticket with easier repro steps

working on tests now

we should have tests for the expression context too

(those tests shouldn't fail today, if my thinking is right)

but it would catch any future regression

+  (let [context (:context env)
+        wrap? (or (= :expr context) (= :statement context))]

:return

thanks very much

failing tests patch submitted. thanks