Not a reviewer.

Just noticed that tests don't have @modules.
Is it better to include @modules when introducing new test?

Thanks,
Amy

On 5/19/17 9:38 AM, Felix Yang wrote:
Ping:)

-Felix
On 16 May 2017, at 4:59 PM, Felix Yang <felix.y...@oracle.com> wrote:

Hi there,

    please review the new tests added for ServiceLoader updates for module 
system.

Test bug:

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

Webrev:

    http://cr.openjdk.java.net/~xiaofeya/8087307/webrev.00/

Related product Changes:

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

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

Thanks,

Felix



Reply via email to