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

--- Comment #6 from Mark Thomas <[email protected]> ---
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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to