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

Semyon Sadetsky semyon.sadetsky at oracle.com
Fri Jan 5 14:58:14 UTC 2018


Hi Pankaj,

Note that before the selection was never cleared if the object is not 
found and the DefaultListSelecetionModel is used. This may cause 
compatibility issue in old applications.

Since the clearing selection behavior is now mentioned in 
setSelectedValue() javadoc it should be also clarified for

setSelectedIndex()
setSelectedIndices()
setSelectionInterval()

and their implementation may need to be changed as well to be consistent.

--Semyon

On 01/05/2018 06:09 AM, Pankaj Bansal wrote:
> Hi Andrej,
>
> Sorry I missed this small thing. Following is the new webrev
> webrev: http://cr.openjdk.java.net/~pbansal/5076761/webrev.04/
>
> Regards,
> Pankaj Bansal
>
> -----Original Message-----
> From: Andrej Golovnin [mailto:andrej.golovnin at gmail.com]
> Sent: Friday, January 5, 2018 6:52 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.03/
> src/java.desktop/share/classes/javax/swing/JList.java
>
> 2379             for(i = 0, c = dm.getSize(); i < c; i++) {
>
> There should be a space after "for". Otherwise looks good. Thanks!
>
> Best regards,
> Andrej Golovnin




More information about the swing-dev mailing list