assert on ttl fn value makes any callable other than a pure clojure.lang.Fn be rejected
Description
ttl memoize used to accept multimethods for instance, this is no longer the case since the latest version where an assert? was introduced that checks using fn?
The patch attached allows any of the following: clojure.lang.IFn, clojure.lang.AFn, java.lang.Runnable, java.util.concurrent.Callable.
(I am a registered contributor listed under "Maximilien Penet (mpenet)")
Environment
None
Attachments
1
Activity
Sean Corfield March 1, 2018 at 7:29 AM
Closing all issues resolved as of release 0.5.8 to clear the decks for future releases.
Fogus June 28, 2013 at 1:31 PM
Applied to master. A new release will come soon.
Fogus June 28, 2013 at 1:17 PM
This is fixed on master. I will push a point-release out to Maven Central later today.
ttl memoize used to accept multimethods for instance, this is no longer the case since the latest version where an assert? was introduced that checks using fn?
The patch attached allows any of the following: clojure.lang.IFn, clojure.lang.AFn, java.lang.Runnable, java.util.concurrent.Callable.
(I am a registered contributor listed under "Maximilien Penet (mpenet)")