RFR: 8039438: Some tests depend on internal API sun.misc.IOUtils
Alan Bateman
Alan.Bateman at oracle.com
Wed Apr 9 18:42:16 UTC 2014
On 09/04/2014 17:00, Alexandre (Shura) Iline wrote:
> Hi.
>
> I'd like to ask for review of this fix:
> http://cr.openjdk.java.net/~shurailine/8039438/webrev.00/
>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8039438
Looks okay to me.
For the class description then maybe something like "Defines useful I/O
methods" might be a bit better than "Holds useful io shortcuts" but it's
a minor point. I guess I would probably make it final and/or a private
constructor to make it obvious that it just defines static methods.
A minor comment on LambdaClassLoaderSerialization.java is that would be
good to indeed the tags so that they are consistent with other tests
(pre-dates your change but more noticable now).
-Alan
More information about the core-libs-dev
mailing list