https://issues.apache.org/bugzilla/show_bug.cgi?id=52549

--- Comment #4 from Mark Thomas <ma...@apache.org> 2012-01-29 17:45:17 UTC ---
Just for the record, you appear to have missed the point of this code.

The list of ServletContainerInitializer is obtained from
META-INF/services/javax.servlet.ServletContainerInitializer within each JAR,
not from scanning the classes and looking for classes that implement it.

The scanning is only done if there is at least one ServletContainerInitializer
that defines HandlesTypes and the scanning is looking for classes that extend
or implement the classes/interfaces defined by HandlesTypes. Loading the class
was a quick and dirty solution (that has survived longer than I thought it
might) to determining if the class meets the extends or implements test.

See the duplicate for my comments on your suggestions. Short version this is
doable with some refactoring and next on my todo list.

Feel free to change the duplicate to a bug if you wish. I'm not that bothered
since it is getting fixed anyway.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- 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