Fork me on GitHub
#cljs-dev
<
2020-07-18
>
dnolen22:07:43

@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

dpsutton22:07:02

Should be attached. Let me look

dnolen22:07:12

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

dpsutton22:07:54

linked in the async ticket now as well

dpsutton23:07:03

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.

dnolen23:07:09

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

dnolen23:07:24

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

dpsutton23:07:09

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