Looks fine.
On 04.08.2020 02:56, Prasanta Sadhukhan wrote:
OK. Please find modified
http://cr.openjdk.java.net/~psadhukhan/6709913/webrev.1/
Regards
Prasanta
On 04-Aug-20 12:22 PM, Sergey Bylokhov wrote:
Hi, Prasanta.
I suggest making the change in the isPopupVisible shorter:
return popup != null && popup.isVisible();
On 29.07.2020 23:18, Prasanta Sadhukhan wrote:
Hi All,
Please review a fix for an issue whereby
javax.swing.plaf.basic.BasicComboBoxUI.isPopupVisible returns
NullPointerException when called from a overridden method of getModel().
If a class extends JComboBox and overrides getModel() and then it calls
isPopupVisible() from the overriden getModel() it gets NullPointerException the
first time.
Because the popup variable in javax.swing.plaf.basic.BasicComboBoxUI is not set
yet.
Proposed fix is to guard against it by doing a null check for the methods which can be
overriden from JComboBox to access "popup" variable.
Bug: https://bugs.openjdk.java.net/browse/JDK-6709913
webrev: http://cr.openjdk.java.net/~psadhukhan/6709913/webrev.0/
Regards
Prasanta
--
Best regards, Sergey.