<Swing Dev> [11] Review Request: JDK-5076761 : JList.setSelectedValue(null, ...) doesn't do anything

Pankaj Bansal pankaj.b.bansal at oracle.com
Fri Jan 5 11:57:58 UTC 2018


Hi Andrej,

Thanks for the review.

I have made changes according to your suggestions. Please have a look.
Webrev: http://cr.openjdk.java.net/~pbansal/5076761/webrev.03/

Regards,
Pankaj Bansal

-----Original Message-----
From: Andrej Golovnin [mailto:andrej.golovnin at gmail.com] 
Sent: Friday, January 5, 2018 4:33 PM
To: Pankaj Bansal
Cc: Semyon Sadetsky; swing-dev at openjdk.java.net; Sergey Bylokhov
Subject: Re: <Swing Dev> [11] Review Request: JDK-5076761 : JList.setSelectedValue(null, ...) doesn't do anything

Hi Pankaj,

> Webrev: http://cr.openjdk.java.net/~pbansal/5076761/webrev.02/

src/java.desktop/share/classes/javax/swing/JList.java

You can simplify the code in lines

2373         if((anObject == selectedValue) ||
2374                 (anObject != null && anObject.equals(selectedValue))) {

and

2383                 if ((anObject == object) ||
2384                         anObject!= null && anObject.equals(object)) {

by using Objects.equals(Object, Object):

And please add spaces in the line 2379 where needed (see http://cr.openjdk.java.net/~alundblad/styleguide/index-v6.html#toc-whitespace
for details).

Best regards,
Andrej Golovnin



More information about the swing-dev mailing list