cljs-dev

ClojureScript compiler & std lib dev, https://clojurescript.org/community/dev
dnolen 2020-07-18T22:53:43.411700Z

@dpsutton hey is there an JIRA ticket w/ your CLJS patch for the and/or work? I found the core.async one but no patch and it didn't link to anything

dpsutton 2020-07-18T22:53:56.412100Z

I think so

dpsutton 2020-07-18T22:54:02.412600Z

Should be attached. Let me look

dnolen 2020-07-18T22:54:12.412900Z

link? Let me assign it myself and I can review and give some feedback

dpsutton 2020-07-18T22:55:02.413100Z

https://clojure.atlassian.net/browse/CLJS-3260

dnolen 2020-07-18T22:55:55.413300Z

thanks!

dpsutton 2020-07-18T22:57:54.413600Z

linked in the async ticket now as well

dnolen 2020-07-18T22:59:37.413800Z

great

dpsutton 2020-07-18T23:03:03.414700Z

I didn't have any prior art for how to turn one ast node into another. I thought i remembered stuff like this in tools analyzer cljs but that repo was marked deprecated.

dnolen 2020-07-18T23:05:09.415400Z

yeah we might need to copy stuff from there or the analyzer for clj

dnolen 2020-07-18T23:05:24.415800Z

if it turns out that this work needs a few AST helpers that we will reuse later that's fine and good

dpsutton 2020-07-18T23:49:09.416900Z

Another thing I had thought of, is it possible that closure might optimize this anyways and the ast transformation isn’t necessary?