RFR JDK-8073924: Update test/java/nio/charset/Charset/NIOCharsetAvailability.java to work with module system
Alan Bateman
Alan.Bateman at oracle.com
Thu Feb 26 21:05:15 UTC 2015
On 26/02/2015 02:47, Xueming Shen wrote:
> Please help review the change for JDK-8073924.
>
> issue: https://bugs.openjdk.java.net/browse/JDK-8073924
> webrev: http://cr.openjdk.java.net/~sherman/8073924/webrev
This looks okay although you should avoid walking the entire file system
with:
Path root1 = fs.getPath("/java.base/sun/nio.cs");
Path root1 = fs.getPath("/jdk.charsets/sun/nio/cs/ext");
Stream.concat(Files.walk(root1), Files.walk(root2))...
An alternative is of course Files.walkFileTree and FileVisitor that
returns SKIP_SUBTREE.
One other thing to be aware if that jrt file system needs to change soon
to have /modules and /packages top level directories. Sundar is working
on that so I guess this test will need to be updated as part of that work.
-Alan
More information about the core-libs-dev
mailing list