From 73b15b70930d0f545a8f23e4125770c0a2d48e9d Mon Sep 17 00:00:00 2001
From: Ryan Doyle <rdoyle@aconex.com>
Date: Fri, 16 Sep 2016 12:00:35 +1000
Subject: [PATCH] Fix non-weak reference regression introduced with Guava
 upgrade

In 0c858c81ed70c36d4ee598eab742221b93b001d5, this map was migrated to
LoadingCache. The weakKeys() builder method accidentally got dropped and
the defaults are to hold strong references. This commit re-introduces the
weakKeys() builder method for the LoadingCache.
---
 .../src/main/java/com/custardsource/parfait/timing/ThreadValue.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/parfait-core/src/main/java/com/custardsource/parfait/timing/ThreadValue.java b/parfait-core/src/main/java/com/custardsource/parfait/timing/ThreadValue.java
index dc07e2b..b5481b5 100644
--- a/parfait-core/src/main/java/com/custardsource/parfait/timing/ThreadValue.java
+++ b/parfait-core/src/main/java/com/custardsource/parfait/timing/ThreadValue.java
@@ -49,7 +49,7 @@ public final T getForThread(Thread thread) {
 
     public static class WeakReferenceThreadMap<T> implements ThreadValue<T> {
 
-        protected final LoadingCache<Thread, T> loadingCache = CacheBuilder.newBuilder().build(new CacheLoader<Thread, T>() {
+        protected final LoadingCache<Thread, T> loadingCache = CacheBuilder.newBuilder().weakKeys().build(new CacheLoader<Thread, T>() {
             @Override
             public T load(Thread thread) throws Exception {
                 return initialValue();