Bug 49510

Summary: Remove dependency on classes copied from commons
Product: Taglibs Reporter: Jeremy Boynes <jboynes>
Component: Standard TaglibAssignee: Tomcat Developers Mailing List <dev>
Severity: enhancement    
Priority: P2    
Version: 1.2.0   
Target Milestone: ---   
Hardware: Macintosh   
OS: All   
Attachments: Patch remove dependencies on LRUMap

Description Jeremy Boynes 2010-06-27 13:47:22 UTC
The "extra" package in standard-impl contains classes copied from Commons Collections, primarily LRUMap used in ElEvaluator and FormatDateSupport. Comments indicate this can be replaced with LinkedHashMap if the baseline is set to Java 1.4 and the project is now using Java5 so these could be removed.
Comment 1 Jeremy Boynes 2010-06-27 13:52:47 UTC
Created attachment 25647 [details]
Patch remove dependencies on LRUMap

Once applied, the project will compile with the extras directory removed completely. There's other things in there that don't appear to be used.
Comment 2 Henri Yandell 2010-06-29 03:38:05 UTC
Thanks Jeremy - patch applied:

svn ci -m "Removing dependency on copied LRUMap class from Commons Collections now that we're 1.5 dependent. Patch courtesy of Jeremy Boynes (#49510)"
Sending        impl/src/main/java/org/apache/taglibs/standard/lang/jstl/ELEvaluator.java
Sending        impl/src/main/java/org/apache/taglibs/standard/tag/common/fmt/FormatDateSupport.java
Transmitting file data ..
Committed revision 958854.