RFR JDK-8073924: Update test/java/nio/charset/Charset/NIOCharsetAvailability.java to work with module system

Xueming Shen xueming.shen at oracle.com
Thu Feb 26 22:09:50 UTC 2015


On 02/26/2015 01:49 PM, Alan Bateman wrote:
> On 26/02/2015 21:23, Xueming Shen wrote:
>>
>> This looks nice :-) webrev has been updated accordingly.
>>
>> http://cr.openjdk.java.net/~sherman/8073924/webrev
> This looks okay but if you want to reduce the code a bit then you could use subpath to drop the module name before mapping it to the string representation ,like this:
>
> .map(p -> p.subpath(1, p.getNameCount()))
> .map(Object::toString)
> .filter(s -> s.endsWith(".class"))
>

thanks! webrev has been updated accordingly.

http://cr.openjdk.java.net/~sherman/8073924/webrev

-sherman



More information about the core-libs-dev mailing list