satisfies? doesn't work with the new instance-based protocol polymorphism

Question

I'd expect that `satisfies?` works even when providing protocol implementation via metadata, however that doesn't seem to be the case:


(defprotocol Foo :extend-via-metadata true (foo [x]))
(foo (with-meta [42] {`foo (fn [x] :boo)}))
;; => :boo

;; but `satisfies?` doesn't work
(satisfies? Foo (with-meta [42] {`foo (fn [x] :boo)}))
;; => false


*Patch*: 0001-CLJ-2426-fix-satisfies-for-new-instance-based-protoc.patch

Comments

Ran into this yesterday.

Answer 1

Comment made by: alexmiller

Would it be better to put the metadata check second instead of first for perf? Or is it actually faster?

Answer 2

Reference: https://clojure.atlassian.net/browse/CLJ-2426 (reported by jumar)