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

--- Comment #5 from Christopher Schultz <ch...@christopherschultz.net> ---
(In reply to Konstantin Kolinko from comment #2)
> Also noted the third issue:
> 
> ImportHandler.importClass() does not allow to import the exactly same class
> twice.
> 
> I think that such imports shall be silently swallowed. It boils down to
> adding a check whether conflicting Class instances are the same, (conflict
> == clazz).

I haven't read through the code to see how class object references are
maintained, but I have been bitten in the past when
Foo.class.getMethod("bar").getName() == Foo.class.getMethod("bar").getName()
yields false.

Perhaps clazz.equals(conflict) would be better than == in this case?

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