Looks fine. +1
It might have been just as effective to add a TEST.properties file with
modules = jdk.localedata
in test/sun/util/resources and test/sun/util/locale directories.
Though the cleanup of the test source is fine too.
Thanks, Roger
On 9/7/2016 10:43 AM, Sergei Kovalev wrote:
Hello team,
Please review small fix in tests source.
Bug ID: https://bugs.openjdk.java.net/browse/JDK-8165604
Review link: http://cr.openjdk.java.net/~skovalev/8165604/webrev.00/
Goal: make test working with --limit-modules ... flag.
Summary: added 'jdk.localedata' into modules declaration section.
