Welcome! Please see the About page for a little more info on how this works.

+1 vote
in ClojureScript by
We have {{\*warn-on-reflection\*}} enabled for the {{cljs.analyzer}} and {{cljs.compiler}} namespaces.

This ticket asks that this also be enabled on other large, performance-critical namespaces, like {{cljs.closure}}, where reasonable.

This would help silly perf issues. For example this one call, when revised to avoid reflection by using {{clojure.string/starts-with?}}, improves the speed of indexing {{node_modules}} by a factor of 2: https://github.com/clojure/clojurescript/blob/132d3aa232921a3cea66f830d61c89be78c581cb/src/main/clojure/cljs/closure.clj#L2464

2 Answers

0 votes
by

Comment made by: dnolen

Sounds good to me!

0 votes
by
Reference: https://clojure.atlassian.net/browse/CLJS-2748 (reported by mfikes)
...