ben-manes / caffeine / #5485

100%

master: 100%

push

github

Honor RemapHints.preserveRefresh in UnboundedLocalCache

The async synchronous view flags query-style no-ops — putIfAbsent on a present
key, a non-matching conditional remove/replace, and a same-instance compute —
with RemapHints.preserveRefresh so they leave an in-flight refresh intact (they
aren't real mutations). BoundedLocalCache.remap honored the hint, but
UnboundedLocalCache.compute dropped it and its remap always discarded, so those
no-ops cancelled a racing refresh(key) on unbounded async caches while the
bounded sibling preserved it.

Thread the hint through UnboundedLocalCache.compute -> remap and skip the
discard on a same-instance return when preserveRefresh is set, mirroring
BoundedLocalCache.remap. A real mutation still discards the refresh.

Found by /audit-sibling-divergence.

4011 of 4018 branches covered (99.83%)

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

8205 of 8205 relevant lines covered (100.0%)

1.0 hits per line