Hi Chris,

Looks good to me as well.

best regards,

-- daniel

On 29/06/2018 07:48, Chris Yin wrote:
Thank you, Vyom

Regards,
Chris

On 29 Jun 2018, at 11:13 AM, vyom tewari <[email protected]> wrote:

looks good to me.

Vyom

On Friday 29 June 2018 06:22 AM, Chris Yin wrote:
Hi, Vyom

Sure, fixed the tag order as you suggested, thanks

New changes:

diff -r 1308189b0848 test/jdk/com/sun/jndi/dns/Test6991580.java
--- a/test/jdk/com/sun/jndi/dns/Test6991580.java        Thu Jun 28 17:45:59 
2018 -0700
+++ b/test/jdk/com/sun/jndi/dns/Test6991580.java        Fri Jun 29 08:48:05 
2018 +0800
@@ -33,10 +33,11 @@
  /*
   * @test
   * @bug 6991580 8080108 8133035
- * @requires os.family != "windows"
   * @summary IPv6 Nameservers in resolv.conf throws NumberFormatException
   * @modules java.desktop
   *          jdk.naming.dns/com.sun.jndi.dns
+ * @requires os.family != "windows"
+ * @build IPv6NameserverPlatformParsingTest
   * @run main/manual Test6991580
   */

Regards,
Chris

Reply via email to