> -----Original Message----- > From: Lukasz Lenart > Sent: Tuesday, June 07, 2011 2:23 > To: Struts Developers List > Subject: Re: org.apache.struts2.util.MakeIterator > > 2011/5/30 Jason Pyeron <jpye...@pdinc.us>: > > In the convert method: > > > > } else if (value instanceof Enumeration) { > > Enumeration enumeration = (Enumeration) value; > > ArrayList list = new ArrayList(); > > > > while (enumeration.hasMoreElements()) { > > list.add(enumeration.nextElement()); > > } > > > > iterator = list.iterator(); > > } else { > > > > > > Why is a wrapper not being used? This is in effect doing double > > iteration, extra memory, etc. > > > > There is a wrapper in org.apache.struts2.util.IteratorFilterSupport: > > EnumerationIterator. > > > > Any reasons it should not be patched? > > Nop, I think it was simple oversight ;-)
https://issues.apache.org/jira/browse/WW-3645 -- -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=- - - - Jason Pyeron PD Inc. http://www.pdinc.us - - Principal Consultant 10 West 24th Street #100 - - +1 (443) 269-1555 x333 Baltimore, Maryland 21218 - - - -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=- This message is copyright PD Inc, subject to license 20080407P00. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org