Fine with me. At first I thought FastHashMap would be needed,
but access to the map ended up being witin other sychnronized code
blocks.
Glenn
costinm@covalent.net wrote:
>
> I was looking at jasper2, and there is one lonely use of FastHashMap in
> compiler/JspRuntimeContext.java. It's the only place where jasper2 depends
> on commons-collections, and it seems pretty useless - nothing calls
> setFast(), and all that happens is an extra ( synchronized ) call to the
> original map.
>
> Any problem if I replace it with a regular HashMap ( or Collections.syncMap(HashMap)
?
>
> Costin
>
> --
> To unsubscribe, e-mail: <mailto:tomcat-dev-unsubscribe@jakarta.apache.org>
> For additional commands, e-mail: <mailto:tomcat-dev-help@jakarta.apache.org>
--
----------------------------------------------------------------------
Glenn Nielsen glenn@more.net | /* Spelin donut madder |
MOREnet System Programming | * if iz ina coment. |
Missouri Research and Education Network | */ |
----------------------------------------------------------------------
--
To unsubscribe, e-mail: <mailto:tomcat-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:tomcat-dev-help@jakarta.apache.org>
|