core.memoize

assert on ttl fn value makes any callable other than a pure clojure.lang.Fn be rejected

Details

  • Type: Defect Defect
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
  • Patch:
    Code

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)")

Activity

Hide
Fogus added a comment -

Applied to master. A new release will come soon.

Show
Fogus added a comment - Applied to master. A new release will come soon.
Hide
Fogus added a comment -

This is fixed on master. I will push a point-release out to Maven Central later today.

Show
Fogus added a comment - This is fixed on master. I will push a point-release out to Maven Central later today.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: