[ASYNC-11] core.async CLJS support incorrect handling of try/catch Created: 19/Jul/13 Updated: 26/Jul/13 Resolved: 26/Jul/13
|Reporter:||David Nolen||Assignee:||Timothy Baldridge|
core.async is defaulting to lein cljsbuild's ClojureScript instead of more recent versions, this has hidden an error with respect to try/catch. core.async try/catch handling emits code like:
This is not correct. ClojureScript requires that you specify the prototype of the error that you want to match. Code like this will result in an error about invalid binding form.
You must emit code that looks like this:
|Comment by Timothy Baldridge [ 19/Jul/13 11:48 AM ]|
lol, I had this code in place, but removed it because it wouldn't compile correctly. This must have changed in a recent version of CLJS.
|Comment by David Nolen [ 19/Jul/13 11:51 AM ]|
I commented out the try/catch tests in master. I'll hand this one over to you
|Comment by Timothy Baldridge [ 26/Jul/13 8:17 AM ]|
Fixed in master