Macro :arglists include &form and &env

Question

Minor issue, but nice to fix especially because of tooling. In lumo:

cljs.user=> (:arglists (get-in @lumo.repl/st [:cljs.analyzer/namespaces 'cljs.core$macros :defs 'when])) ([&form &env test & body])

I would like to filter out the first two arguments that are of course not needed.
A simple patch is (link: https://github.com/anmonteiro/lumo/pull/88/files#diff-4da6b6777b07770bfe3ffbd39a773e33R823 text: here).

I can provide a patch once I figure out where is the right place to apply it. Of course if I am not doing it wrong.

Thanks!

Answer 1

Comment made by: arichiardi

On second thought, if we put this filter in load-analysis-cache!, probably we will pay to much of a price for just a display/ide problem. The best thing I can think of is to filter the transit/json file before loading. So probably this is better to be inside lumo.

Answer 2

Comment made by: arichiardi

After conversation with (link: ~anmonteiro): this inconsistency might be solved with (link: http://dev.clojure.org/jira/browse/CLJS-1461 text: CLJS-1461) so this one here can be closed I guess.

Answer 3

Comment made by: mfikes

Perhaps related to CLJS-2852

Answer 4

Reference: https://clojure.atlassian.net/browse/CLJS-1958 (reported by arichiardi)