[9] RFR: 8166645: Include locales plugin throws InternalError with "*" specified.
Mandy Chung
mandy.chung at oracle.com
Tue Oct 4 02:53:52 UTC 2016
> On Oct 3, 2016, at 3:17 PM, Naoto Sato <naoto.sato at oracle.com> wrote:
>
> According to an internal review, I added some explanation to the modification. No logic modifications.
>
> http://cr.openjdk.java.net/~naoto/8166645/webrev.01/
This looks okay.
I suggest to keep Set<String> originals after splitting the input byte array such that
325 .filter(t -> original.indexOf(t) != -1)
can be replaced with .filter(originals::contains)
Mandy
More information about the core-libs-dev
mailing list