On 2/14/14 10:31 AM, Michael McMahon wrote:
On 14/02/14 18:20, Alan Bateman wrote:
On 14/02/2014 17:42, Michael McMahon wrote:
Could I get the following change reviewed please?

http://cr.openjdk.java.net/~michaelm/8034853/webrev.1/

The change is to remove the class sun.misc.ClassLoaderUtil.
The functionality provided by this class is now in the public
java.net.URLClassloader.close() method.
The removal is good. Maybe the test should be moved to URLClassLoader as it doesn't make sense to have it in ClassLoaderUtil anymore.

-Alan.

I think I'd probably just delete the test since URLClassLoader.close has its own tests,

The change looks good to me. I think URLClassLoader/closetest test covers this case already. I agree with Alan that you can simply delete this old test.

Mandy

Reply via email to