[Bug 57132] ImportHandler.resolveClass() fails to report conflicting import when called repeatedly

2014-10-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57132 --- Comment #6 from Konstantin Kolinko --- (In reply to Christopher Schultz from comment #5) > > 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.getMe

[Bug 57132] ImportHandler.resolveClass() fails to report conflicting import when called repeatedly

2014-10-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57132 --- Comment #5 from Christopher Schultz --- (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 impo

[Bug 57132] ImportHandler.resolveClass() fails to report conflicting import when called repeatedly

2014-10-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57132 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug 57132] ImportHandler.resolveClass() fails to report conflicting import when called repeatedly

2014-10-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57132 --- Comment #3 from Mark Thomas --- Reading the Javadoc for the second issue, I think returning null is a better response here. I've committed a couple of test cases and a fix for this. Looking at the third issue now... -- You are receiv

[Bug 57132] ImportHandler.resolveClass() fails to report conflicting import when called repeatedly

2014-10-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57132 Konstantin Kolinko changed: What|Removed |Added OS||All --- Comment #2 from Konst

[Bug 57132] ImportHandler.resolveClass() fails to report conflicting import when called repeatedly

2014-10-22 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57132 --- Comment #1 from Konstantin Kolinko --- Created attachment 32139 --> https://issues.apache.org/bugzilla/attachment.cgi?id=32139&action=edit 2014-10-22_tc8_57132_testcases.patch Updated testcases for TestImportHandler to test that dupl