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

+1 vote
in ClojureScript by

If you extend a type to implement a protocol, and then create an instance of that type that carries its own implementation of the same protocol in its metadata, ClojureScript will still use the implementation from the type, whereas Clojure uses the more specific metadata implementation.

Here's a brief example based on one of the tests for ClojureScript's protocol metadata implementation.

Edit: I am unable to make the code sample presentable in this editor, please see this gist instead: https://gist.github.com/cjohansen/a24257ecb5db15c7e20aaa25ff713b30

(defprotocol ExtMetaProtocol
:extend-via-metadata true
(ext-meta-protocol [x]))

(ext-meta-protocol (with-meta {} {`ext-meta-protocol (fn [_] 1)})) ;;=> 1

(extend-type clojure.lang.PersistentArrayMap
ExtMetaProtocol
(ext-meta-protocol [m]

2))

(ext-meta-protocol {}) ;;=> 2
(ext-meta-protocol (with-meta {} {`ext-meta-protocol (fn [_] 1)})) ;;=> cljs => 2, clj => 1

1 Answer

+1 vote
by

Confirmed that Clojure 1.10.1 and ClojureScript 1.10.520 behave differently from each other, in the way described in the sample REPL session. For cljs you must replace extend-type clojure.lang.PersistentArrayMap with extend-type cljs.core.PersistentArrayMap.

I would recommend going into the #cljs-dev channel of Clojurians Slack and pointing the ClojureScript developers at this issue to see if they are already aware of it or not.

by
I implemented the initial :extend-via-metadata support and can confirm that I implemented it based on the wrong assumptions (ie. actual impl is picked before meta).

The tests even reflect that assumption:
https://github.com/clojure/clojurescript/blob/d64b2333f3127b51dbe410874a7cdff4bac1edf8/src/test/cljs/cljs/core_test.cljs#L1756-L1777
by
Do you know whether this is considered a bug in the JavaScript implementation, worth changing?
by
It should probably match the Clojure behavior yes.

It does hoewever mean that protocols using :extend-via-metadata will get "slower" overall since it cannot take the "shortcut" it currently takes.
...