svn commit: r548299 - /incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java

[al...@apache.org]

Author: almaw
Date: Mon Jun 18 03:48:27 2007
New Revision: 548299

URL: http://svn.apache.org/viewvc?view=rev&rev=548299
Log:
Back out to prev.

Modified:
    incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java

Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java?view=diff&rev=548299&r1=548298&r2=548299
==============================================================================
--- incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java (original)
+++ incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java Mon Jun 18 03:48:27 2007
@@ -16,7 +16,6 @@
  */
 package org.apache.wicket.util.lang;
 
-import java.lang.ref.WeakReference;
 import java.lang.reflect.Array;
 import java.lang.reflect.Field;
 import java.lang.reflect.InvocationTargetException;
@@ -272,15 +271,14 @@
 
 	private final static IGetAndSet getGetAndSetter(String exp, Class clz)
 	{
-		Map getAndSetters = (Map)classesToGetAndSetters.get(clz.getName());
+		Map getAndSetters = (Map)classesToGetAndSetters.get(clz);
 		if (getAndSetters == null)
 		{
 			getAndSetters = new ConcurrentHashMap(8);
-			classesToGetAndSetters.put(clz.getName(), getAndSetters);
+			classesToGetAndSetters.put(clz, getAndSetters);
 		}
 
-		WeakReference ref = (WeakReference)getAndSetters.get(exp);
-		IGetAndSet getAndSetter = ref == null ? null : (IGetAndSet)ref.get();
+		IGetAndSet getAndSetter = (IGetAndSet)getAndSetters.get(exp);
 		if (getAndSetter == null)
 		{
 			Method method = null;
@@ -411,7 +409,7 @@
 				field = findField(clz, exp);
 				getAndSetter = new MethodGetAndSet(method, field);
 			}
-			getAndSetters.put(exp, new WeakReference(getAndSetter));
+			getAndSetters.put(exp, getAndSetter);
 		}
 		return getAndSetter;
 	}
@@ -1143,10 +1141,6 @@
 		 */
 		public Method getSetter()
 		{
-			if (setMethod == null)
-			{
-				setMethod = findSetter(getMethod, getMethod.getReturnType());
-			}
 			return setMethod;
 		}