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

Vyom Tiwari vyommani at gmail.com
Fri Mar 27 04:08:58 UTC 2020


Hi Chris,

I have one question to you, is there is any specific reason for using
wildcard(?) ?.
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;
#####################################################################

thanks,
Vyom

On Wed, Mar 25, 2020 at 1:28 PM Chris Yin <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
> Webrev: 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


More information about the core-libs-dev mailing list