Details
-
Type:
Enhancement
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Completed
-
Affects Version/s: None
-
Fix Version/s: None
-
Component/s: None
-
Labels:
-
Patch:Code
Description
Currently during a lookup (without default value) on the SoftCache we'll use the result of (.get r) for the ::nil-check and then get it again as we're about to return it. It's possible this could result in inconsistency but it's also an unnecessary call.
I've provided a patch which reuses the result of the get for the check and return value.
Thanks Neil. I'll look at this (and
CCACHE-52) when I get back from Conj. I don't see you listed here https://clojure.org/community/contributors – have you signed the Contributor's Agreement? (site updates tend to lag behind a bit).CCACHE-52) when I get back from Conj. I don't see you listed here https://clojure.org/community/contributors – have you signed the Contributor's Agreement? (site updates tend to lag behind a bit).