You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by hl...@apache.org on 2006/12/17 21:38:50 UTC

svn commit: r488044 - /tapestry/tapestry5/tapestry-ioc/trunk/src/main/java/org/apache/tapestry/ioc/internal/services/TypeCoercerImpl.java

Author: hlship
Date: Sun Dec 17 12:38:49 2006
New Revision: 488044

URL: http://svn.apache.org/viewvc?view=rev&rev=488044
Log:
Remove ArrayWrapper, as it is not necessary (the way InheritanceSearch defines inheritance)

Modified:
    tapestry/tapestry5/tapestry-ioc/trunk/src/main/java/org/apache/tapestry/ioc/internal/services/TypeCoercerImpl.java

Modified: tapestry/tapestry5/tapestry-ioc/trunk/src/main/java/org/apache/tapestry/ioc/internal/services/TypeCoercerImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/tapestry-ioc/trunk/src/main/java/org/apache/tapestry/ioc/internal/services/TypeCoercerImpl.java?view=diff&rev=488044&r1=488043&r2=488044
==============================================================================
--- tapestry/tapestry5/tapestry-ioc/trunk/src/main/java/org/apache/tapestry/ioc/internal/services/TypeCoercerImpl.java (original)
+++ tapestry/tapestry5/tapestry-ioc/trunk/src/main/java/org/apache/tapestry/ioc/internal/services/TypeCoercerImpl.java Sun Dec 17 12:38:49 2006
@@ -111,15 +111,6 @@
     {
         Defense.notNull(targetType, "targetType");
 
-        // Because say new int[3] and new String[3] have different class objects, we
-        // need to unify them for easy lookup. So wrap the array into a wrapper and
-        // use the class object of the wrapper as the key.
-        // if (input instanceof Object[])
-        // {
-        // input = new ArrayWrapper((Object[]) input);
-        // }
-        // // Treat null as void in terms of locating a coercion.
-
         Class sourceType = input != null ? input.getClass() : void.class;
 
         // The caller may ask for the value in a primitive type, but the best we can do is the