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.golov...@gmail.com] Sent: Friday, January 5, 2018 6:52 PM To: Pankaj Bansal Cc: Semyon Sadetsky; swing-dev@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