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

--- Comment #4 from Nick Williams <nicho...@nicholaswilliams.net> ---
First, here's the companion Spring Framework enhancement request:
https://jira.springsource.org/browse/SPR-10788

/I/ don't need to extend WebappClassLoader. I plan on simply adding to the
existing WebappClassLoader. Spring Framework extends it because that's the only
way to make it work without changing Tomcat directly. The reason any changes
are needed at all is because a standard ClassLoader does not just support
instrumentation out-of-the-box. You must instrument a class file /as it is
being loaded/, before it is initialized/instantiated.

I like the idea of creating registerTransformer() and unregisterTranformer()
instead of addTransformer(), but that is a deviation from the unofficial
convention that other containers abide by. If Mark is also okay with
registerTransformer() and unregisterTranformer(), I'll take that approach.

I'm not exactly sure what getThrowawayClassLoader() is for, either. I have
asked some Spring folks to clarify. It exists consistently across containers
and Spring container-support classes, so I assume it must be important. We'll
see.

-- 
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