+1

Naoto

On 9/7/16 7:22 AM, Sergei Kovalev wrote:
Fixed. Please find new version here:
http://cr.openjdk.java.net/~skovalev/8165592/webrev.01/


07.09.16 16:13, Alexandre (Shura) Iline wrote:
"Copyright (c) 2007,2016 Oracle”

Should be

"Copyright (c) 2007, 2016, Oracle”

Otherwise good.

Shura

On Sep 7, 2016, at 5:41 AM, Sergei Kovalev
<[email protected]> wrote:

Hello team,

Please review small fix for tests issue:

Bug ID: https://bugs.openjdk.java.net/browse/JDK-8165592
Webrev:
http://cr.openjdk.java.net/~skovalev/8165592/webrev.00/index.html

Summary: added 'jdk.localedata' into modules declaration section to
have an ability to skip the tests with --limit-modules execution if
required. Also made some import clean up.

The changes tested locally using jtreg tool.

--
With best regards,
Sergei


Reply via email to