From b4eb810347ac073c29b62bb0fc2c70c40376aafb Mon Sep 17 00:00:00 2001
From: Joel Leitch <joel.leitch@gmail.com>
Date: Wed, 6 Apr 2011 00:02:40 +0000
Subject: [PATCH] Comments for r766.

---
 .../com/google/gson/DefaultConstructorAllocator.java |  4 ++--
 .../google/gson/DefaultConstructorAllocatorTest.java | 12 +++---------
 2 files changed, 5 insertions(+), 11 deletions(-)

diff --git a/gson/src/main/java/com/google/gson/DefaultConstructorAllocator.java b/gson/src/main/java/com/google/gson/DefaultConstructorAllocator.java
index b8df6407..5837fcc8 100644
--- a/gson/src/main/java/com/google/gson/DefaultConstructorAllocator.java
+++ b/gson/src/main/java/com/google/gson/DefaultConstructorAllocator.java
@@ -38,8 +38,8 @@ final class DefaultConstructorAllocator {
   }
 
   // for testing purpose
-  final Cache<Class<?>, Constructor<?>> getConstructorCache() {
-    return constructorCache;
+  final boolean isInCache(Class<?> cacheKey) {
+    return constructorCache.getElement(cacheKey) != null;
   }
 
   private static final Constructor<Null> createNullConstructor() {
diff --git a/gson/src/test/java/com/google/gson/DefaultConstructorAllocatorTest.java b/gson/src/test/java/com/google/gson/DefaultConstructorAllocatorTest.java
index 820e79dc..c20443b8 100644
--- a/gson/src/test/java/com/google/gson/DefaultConstructorAllocatorTest.java
+++ b/gson/src/test/java/com/google/gson/DefaultConstructorAllocatorTest.java
@@ -16,8 +16,6 @@
 
 package com.google.gson;
 
-import com.google.gson.DefaultConstructorAllocator;
-
 import junit.framework.TestCase;
 
 import java.util.ArrayList;
@@ -41,20 +39,16 @@ public class DefaultConstructorAllocatorTest extends TestCase {
   public void testObjectConstructor() throws Exception {
     ArrayList<Object> arrayList = allocator.newInstance(ArrayList.class);
     assertTrue(arrayList.isEmpty());
-    assertInCache(ArrayList.class);
+    assertTrue(allocator.isInCache(ArrayList.class));
 
     LinkedList<Object> linkedList = allocator.newInstance(LinkedList.class);
     assertTrue(linkedList.isEmpty());
-    assertInCache(LinkedList.class);
+    assertTrue(allocator.isInCache(LinkedList.class));
   }
 
   public void testMissingDefaultConstructor() throws Exception {
     assertNull(allocator.newInstance(NoDefaultConstructor.class));
-    assertInCache(NoDefaultConstructor.class);
-  }
-
-  private void assertInCache(Class<?> clazz) {
-    assertNotNull(allocator.getConstructorCache().getElement(clazz));
+    assertTrue(allocator.isInCache(NoDefaultConstructor.class));
   }
 
   private static class NoDefaultConstructor {