Github user koraktor commented on the issue:
https://github.com/apache/tomcat/pull/108
@markt-asf @ChristopherSchultz
Thanks for your feedback so far and for the partial merge. To be honest I
didnât think about any performance implications or side effects because I
didnât notice that this could be used in other places, especially during each
request.
Iâm sorry for the broken second patch and pushed an updated version which
fixes the unused imports in `HostConfig` and broken syntax in `ContextName`.
Iâll have another look at the regex precompilation and Java 7
compatibility later. After that I want to look at improving performance and
backwards-compatibility. Maybe someone can point me at the right code locations
where this might be important?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
