This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2017-06-02
Channels
- # aleph (5)
- # beginners (112)
- # boot (137)
- # cider (10)
- # cljs-dev (36)
- # cljsrn (2)
- # clojure (118)
- # clojure-argentina (1)
- # clojure-berlin (1)
- # clojure-brasil (3)
- # clojure-dev (4)
- # clojure-italy (2)
- # clojure-nl (13)
- # clojure-russia (23)
- # clojure-spec (5)
- # clojure-uk (53)
- # clojurescript (344)
- # clojutre (1)
- # core-async (65)
- # cursive (9)
- # datascript (7)
- # datomic (28)
- # devops (1)
- # emacs (16)
- # events (1)
- # jobs (5)
- # keechma (18)
- # lumo (56)
- # off-topic (7)
- # om (3)
- # onyx (14)
- # protorepl (21)
- # re-frame (3)
- # reagent (20)
- # ring (12)
- # ring-swagger (9)
- # specter (17)
- # unrepl (14)
- # vim (14)
- # yada (22)
Should there be an ns
macro defined in core, for the sole purpose of providing a doc string?
Oh I see, so it's working in other editors and in the REPL (Which I never use directly by hand). Cursive doing static analysis isn't able to pick it up.
Ahh, right. Probably something Colin can special-case in the ClojureScript REPL support.
I guess it's Cursive issue then. Which should (maybe at some point) do dynamic doc lookup. Similar to the generate stubs for CLJ.
Cursive has a GitHub set up for issues; Colin has been focusing on improved ClojureScript support
Yeah I've reported a ton of issue there. I'll wait... no need for special case of ns
for now. Was asking since I didn't realize it already had a proper docstring.
@mfikes I’m also seeing big gains for your method analysis patch esp. if I do so JVM memory settings tweaks
It was motivated by the known inexplicable slowness of for
, but perhaps there is enough of that kind of stuff going on for it to add up
Yeah... I feel like ClojureScript has somewhat matured and now we are working on icing on the cake 🙂
@rauh Do you generally blog about Clojure(Script). I'm thinking about a short post about these perf improvements and the theme of maturity/perf optimizations, and would mention your patch if you don't plan to.
@mfikes I don't. And please don't link to my website, I should just take that thing down 😄 Mentioning me by name is fine.
I’m also loving these perf patches go in while I watch from the sidelines
thanks for your work @rauh
FWIW, there is still another easy 20% - 30% for grabs with the shadowing thing by writing it so it's always a single map lookup. Thus maintaining a second map that keeps all those prefixes of the known namespaces. Least hacky would be to just always add the prefix right when the namespace is added to the compiler state.
Nope... they are what the compiler spits out. I re-ran them a few times after that, of course, to ensure they were not anomalies. They are very reproducible. 🙂 And yes. 2×, baby!