Hi! I propose to drop the "org.apache.tomcat.util.collections" package in our trunk (TC7).
It contains the following classes: MultiMap MultiMapNamesEnumeration MultiMapValuesEnumeration Rationale: Cleanup. The implementation is incomplete, e.g. MultiMap$Field.name and .value fields are never assigned. This situation is the same in trunk, TC6, TC5.5. As I see from comments, the MultiMap class was intended to provide a faster implementation for org.apache.tomcat.util.http.Parameters, MimeHeaders, Cookies. That never happened. As of 5.5, 6.0 and trunk the MimeHeaders, Cookies classes already have their own "fast" implementation and won't use this class. The Parameters class still uses a Hashtable, but I doubt that there is a need to make its implementation faster. In r934968 I removed the dependency between Parameters and MultiMap in our trunk. There were some other classes in the "org.apache.tomcat.util.collections" package in the 5.5, 6.0 versions of Tomcat, but those have been removed already. That said, if anybody objects the removal and sees some purpose for keeping that MultiMap class, I will leave it where it is and will just mark it as deprecated. Best regards, Konstantin Kolinko --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
