RFR: JDK-8155649: IncludeLocalesPlugin throws NPE when jdk.localedata not resolved

[Naoto Sato]

Thanks, Alan.

I thought of the same for en_US w/o jdk.localedata case, and reached the 
same conclusion.

I found one typo in my fix, where "-addmods" in plugins.properties 
should be "--addmods". I will correct it and push the changeset.

Naoto

On 5/4/16 3:03 AM, Alan Bateman wrote:
> On 04/05/2016 00:33, Naoto Sato wrote:
>> Hi,
>>
>> Please review the fix for the following issue:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8155649
>>
>> The proposed fix is located at:
>>
>> http://cr.openjdk.java.net/~naoto/8155649/webrev.00/
> This looks okay. There is a corner case where it should be possible to
> --include-locales to specify en_US and it should always succeed but I
> don't think that is worth spending time on.
>
> -Alan
>