Bug 49510

Summary:	Remove dependency on classes copied from commons
Product:	Taglibs	Reporter:	Jeremy Boynes <jboynes>
Component:	Standard Taglib	Assignee:	Tomcat Developers Mailing List <dev>
Status:	RESOLVED FIXED
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.