Fork me on GitHub
#cljs-dev
<
2022-05-13
>
dnolen15:05:00

@borkdude did you open an issue for the metafn thing?

borkdude15:05:09

@dnolen Sorry I didn't follow up on that, but it wasn't a bug I could reproduce. I thought I reproduced it. but that was just a limitation of protocols not being able to support varargs, so ((MetaFn. (fn [& xs])) 0 .. 22) is never going to work. I did run into this while trying to apply the MetaFn as well, but I had trouble reproducing it and since I had a workaround (going via (.-afn the-meta-fn)), I gave up.

dnolen15:05:24

ok, so I think this is not related to the other thing unless you checked this was a regression

borkdude15:05:35

I don't think it was a regression

dnolen15:05:43

but yeah static invokes for 20+ args need compiler support, and a ticket exists for that

dnolen15:05:53

the patch applied only fixed .apply

borkdude15:05:31

In my code, I had (apply f x1 x2 xs) and this failed when xs was longer than 18-20-ish elements

borkdude15:05:40

but this was not a regression with CLJS master

borkdude15:05:08

and f was a MetaFn.

dnolen15:05:46

ok, probably in the next iteration will look at finally making all this consistent

borkdude15:05:04

that's great and I'm happy with the workaround, since it will support older CLJS versions too

dnolen16:05:45

feedback on the notes welcome

💖 1
🎉 1
dnolen20:05:19

@alexmiller can we rebuild the website?