On 23/01/2018 07:01, Chris Yin wrote:
Please review the added JNDI test javax/naming/dns/AttributeTests/GetAny.java, 
thanks


You may want to move it to com/sun/jndi/dns so that it's with the other tests for the DNS provider (as there is no javax.naming.dns API). Also I suspect you don't need "@modules jdk.naming.dns/com.sun.jndi.dns" as it doesn't appear to make direct use of the classes in the implementation.

-Alan

Reply via email to