[15] RFR: 8214694: cleanup rawtypes warnings in open jndi tests

[Chris Yin]

Hi, Vyom

> On 27 Mar 2020, at 12:08 PM, Vyom Tiwari <vyommani at gmail.com> wrote:
> 
> Hi Chris,
> 
> I have one question to you, is there is any specific reason for using wildcard(?) ?. 

Thank you for reviewing and comments. I just replaced most of the wildcard(?) with specified type as precise as they could be in latest webrev.01, the rest of them may fall into below scenarios.

1. API return value or parameter with wildcard(?), such as Hashtable<?,?> in test/jdk/com/sun/jndi/dns/EnvTests/AddInherited.java
2. Cannot find the precise type from code, such as ScheduledFuture<?> in test/jdk/com/sun/jndi/ldap/DeadSSLLdapTimeoutTest.java

> In your change we can avoid  "?" at most of the places. Please see the below methods signatures.
> 
> #######################################################################
> public NamingEnumeration<Binding> listBindings(Name name)  throws NamingException;
> public NamingEnumeration<NameClassPair> list(Name name)  throws NamingException;
> public NamingEnumeration<SearchResult>    search(Name name,  Attributes matchingAttributes,
>                                                                                            String[] attributesToReturn)  throws NamingException;
> #####################################################################

Thank you for the detailed signatures info, yes, now all fixed in the latest webrev http://cr.openjdk.java.net/~xyin/8214694/webrev.01/

Regards,
Chris

> 
> thanks,
> Vyom    
> 
> On Wed, Mar 25, 2020 at 1:28 PM Chris Yin <xu.y.yin at oracle.com <mailto:xu.y.yin at oracle.com>> wrote:
> Hello
> 
> Please review following simple changes to cleanup raw types warning for open jndi tests (under test/jdk/com/sun/jndi and test/jdk/javax/naming), thanks
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8214694 <https://bugs.openjdk.java.net/browse/JDK-8214694>
> Webrev: http://cr.openjdk.java.net/~xyin/8214694/webrev.00/ <http://cr.openjdk.java.net/~xyin/8214694/webrev.00/>
> 
> 
> The changes should be straightforward, only fix raw types warnings, no test logic change, no code optimization or cleanup. Minor change to each test file, just a little surprised about the affected tests count, hope this covers all. Run related jndi tests on 4 platforms for total 200 times, all passed.
> 
> Thanks,
> Chris
> 
> 
> -- 
> Thanks,
> Vyom