Widen cljs.core/defprotocol sig arguments from list to seq (to include Cons)

Description

First logged at https://ask.clojure.org/index.php/9092/does-cljs-core-defprotocol-sigs-arguments-could-also-be-cons

For the second time I’ve been bitten by this line:

https://github.com/clojure/clojurescript/blob/master/src/main/clojure/cljs/core.cljc#L2043

Could we replace it with (core/seq? head) ?

This way, when emiting a defrecord form from a macro, sigs could be of type Cons which seems to make sense.

Environment

None

Assignee

Unassigned

Reporter

Alex Miller

Labels

None

Approval

None

Patch

None

Priority

Major
Configure