DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=34849>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=34849 [EMAIL PROTECTED] changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |1.2.5 Milestone ------- Additional Comments From [EMAIL PROTECTED] 2005-05-27 00:59 ------- I'm against adding a dependency on Commons Collections, because of the incompatibility problems they created between 2.1 and 3.0 and because IteratorEnumeration (http://tinyurl.com/98qgt) is such aminor piece of functionality. Also theres an existing alternative solution - we already have an IteratorAdapter in the utils package - changing the method [i.e. PageContextImpl.getAttributeNamesInScope()] to return an Iterator rather than an Enumeration would mean there would be no need to add/copy any additonal classes to struts. I haven't looked in detail, but I believe that switching to an Iterator also then removes the jstl dependency? On the dependency on Commons Lang - the two utility classes are just stored in the PageContext - all the validator does is instantiate them, so I am against creating a dependecny here and would prefer Lukasz's suggestion of just using reflection to instantiate these objects. -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
