Is recur across try supported in ClojureScript?

I found an example that it is not: https://dev.clojure.org/jira/browse/CLJS-2476

(and should arguably fail compilation as is done in Clojure)

@mfikes it shouldn’t work no

Cool. Fortunately, an extremely easy patch is possible by simply adding disallowing-recur.

I'm putting that together, and with three changes can catch all of these problematic cases

(loop [] (try (recur)))
(loop [] (try (catch js/Error t (recur))))
(loop [] (try (catch :default t (recur))))
(loop [] (try (finally (recur))))

k, with tests for those cases?

Yep, those are the test cases.

excellent

Wow. It caught something interesting:

(loop [] (binding [*foo* 1] (recur)))

@dnolen With respect to this bit ^ about binding. We actually (incorrectly) use this construct here https://github.com/clojure/clojurescript/blob/master/src/main/cljs/cljs/js.cljs#L707 Would you prefer that a fix for that problematic recur be done as a separate patch, or combined with the one I'm putting together for https://dev.clojure.org/jira/browse/CLJS-2476 ?

@mfikes ok to fix that in the same patch