Re: [9] RFR: 8075545: Add permission check for locale service provider implementations

2015-04-29 Thread Alan Bateman 

On 28/04/2015 22:59, Naoto Sato wrote:

Hi,

Please review the changes for the following bug:

https://bugs.openjdk.java.net/browse/JDK-8075545

The fix is to check the RuntimePermission target 
"localeServiceProvider" on instance creation. The fix is located at:


http://cr.openjdk.java.net/~naoto/8075545/

This looks okay to me too.

-Alan

RFR: 8077685: (tz) Support tzdata2015d

2015-04-29 Thread Aleksej Efimov 

Hi,

Please, review the fix for adding the latest tzdata2015d to JDK9.
The JTREG and JPRT tests were executed.
No failures were observed.

Thank you,
Aleksej

[1] JBS: https://bugs.openjdk.java.net/browse/JDK-8077685
[2] Webrev: http://cr.openjdk.java.net/~aefimov/tzdata/2015d/9/00

Re: RFR: 8077685: (tz) Support tzdata2015d

2015-04-29 Thread Masayoshi Okutsu 

Looks good to me.

Masayoshi

On 4/29/2015 11:37 PM, Aleksej Efimov wrote:

Hi,

Please, review the fix for adding the latest tzdata2015d to JDK9.
The JTREG and JPRT tests were executed.
No failures were observed.

Thank you,
Aleksej

[1] JBS: https://bugs.openjdk.java.net/browse/JDK-8077685
[2] Webrev: http://cr.openjdk.java.net/~aefimov/tzdata/2015d/9/00