ben-manes / caffeine / #5613

100%

master: 100%

push

github

Preserve timestamps on a raced same-instance refresh

The automatic refreshIfNeeded path checked for an equal-value reload
before the ABA/ownership check and returned without preserving the
timestamps, so a reload completing after a concurrent put(k,
sameInstance) did a full update — shifting the entry's expiration
forward by the reload's in-flight duration and stomping the write.
Mirror the explicit-refresh fix: check ownership first. An owned
reload installs the value (refreshing metadata even for a same
instance), while a raced reload preserves the write's timestamps and
only notifies REPLACED when the value differs.

4125 of 4134 branches covered (99.78%)

3 of 3 new or added lines in 1 file covered. (100.0%)

8388 of 8388 relevant lines covered (100.0%)

1.0 hits per line