fixed as suggested. the similar code exists in java.net.URLClassLoader.
Ah, something in the back of my mind which I didn't mention.
This class is designed to run and compile for old VMs, annotations are not allowed. I'll remove them for you - it was just overrides that you added.
Ok, on topic for the class loader bug....
Seems fine, please integrate.
Well, first comment, can we fix the spelling of that method? :(
I assume it was supposed to be onChangedTestResult(). For compatibility, could you please leave but deprecate the old method and just make it call the new method onChangedTestResult().
On second thought, since that method is an observer method which we expect people to override...
Perhaps just deprecate? And call both when notifications are sent? What do you think?
your comments are about the other bug: 6484939: CustomTestResultViewer, please see my thoughts there
Your use of this web site or any of its content or software indicates your agreement to be bound by these Terms of Participation.
Copyright © 2014, Oracle and/or its affiliates. All rights reserved. Oracle and Java are registered trademarks of Oracle and/or its affiliates. Other names may be trademarks of their respective owners.