Sylvain, This change does not make much sense IMHO. Why do you check for ArrayList instead of List. There is only a List cast after that, so why not just check for List? Is there anything a LinkedList does not correctly implement from the List interface?!
-Manfred 2005/7/26, [EMAIL PROTECTED] <[EMAIL PROTECTED]>: > Author: svieujot > Date: Tue Jul 26 08:25:26 2005 > New Revision: 225327 > > URL: http://svn.apache.org/viewcvs?rev=225327&view=rev > Log: > Bugfix for savestate with a linkedlist (couldn't coerce a value of type > "java.util.ArrayList" to type "java.util.LinkedList". > > Modified: > myfaces/api/trunk/src/java/javax/faces/component/UIComponentBase.java > > Modified: > myfaces/api/trunk/src/java/javax/faces/component/UIComponentBase.java > URL: > http://svn.apache.org/viewcvs/myfaces/api/trunk/src/java/javax/faces/component/UIComponentBase.java?rev=225327&r1=225326&r2=225327&view=diff > ============================================================================== > --- myfaces/api/trunk/src/java/javax/faces/component/UIComponentBase.java > (original) > +++ myfaces/api/trunk/src/java/javax/faces/component/UIComponentBase.java Tue > Jul 26 08:25:26 2005 > @@ -593,9 +593,9 @@ > Object attachedObject) > { > if (attachedObject == null) return null; > - if (attachedObject instanceof List) > + if (attachedObject instanceof ArrayList) > { > - ArrayList lst = new ArrayList(((List)attachedObject).size()); > + List lst = new ArrayList(((List)attachedObject).size()); > for (Iterator it = ((List)attachedObject).iterator(); > it.hasNext(); ) > { > lst.add(saveAttachedState(context, it.next())); > > >
