From dcab15ebed8c08a13faa81da057bef45faa54a10 Mon Sep 17 00:00:00 2001
From: Edwin Kempin <edwin.kempin@sap.com>
Date: Thu, 22 Aug 2013 15:33:34 +0200
Subject: [PATCH] Use CacheBuilder returned from
 CacheBuilder.removalListener(...)

The JavaDoc of CacheBuilder.removalListener(...) states that after
invoking this method this CacheBuilder should not be used anymore but
only the returned CacheBuilder.

Change-Id: Ibe7cb90624ca29e3492dbc84d029758bed0055c3
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
---
 .../com/google/gerrit/server/cache/h2/DefaultCacheFactory.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gerrit-cache-h2/src/main/java/com/google/gerrit/server/cache/h2/DefaultCacheFactory.java b/gerrit-cache-h2/src/main/java/com/google/gerrit/server/cache/h2/DefaultCacheFactory.java
index 5a600c0025..1c850d1ab4 100644
--- a/gerrit-cache-h2/src/main/java/com/google/gerrit/server/cache/h2/DefaultCacheFactory.java
+++ b/gerrit-cache-h2/src/main/java/com/google/gerrit/server/cache/h2/DefaultCacheFactory.java
@@ -85,7 +85,7 @@ public class DefaultCacheFactory implements MemoryCacheFactory {
         "cache", def.name(), "memoryLimit",
         def.maximumWeight()));
 
-    builder.removalListener(forwardingRemovalListenerFactory.create(def.name()));
+    builder = builder.removalListener(forwardingRemovalListenerFactory.create(def.name()));
 
     Weigher<K, V> weigher = def.weigher();
     if (weigher != null && unwrapValueHolder) {