[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