https://bz.apache.org/bugzilla/show_bug.cgi?id=52952

--- Comment #6 from Mark Thomas <ma...@apache.org> ---
Some feedback on the patch.

Objects is Java 7+. Tomcat 7 must run on Java 6 so that part of the patch needs
re-work.

Otherwise the patch looks OK for what it does. What it misses is that there may
be multiple Tomcat instances in an embedded scenario hence my comment about
that rather than being static, ExtensionValidator needed to move to the Server
so there could - in theory at least - be multiple independent instances in a
JVM.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to