<Swing Dev> [8] Review request for 6337518: Null Arrow Button Throws Exception in BasicComboBoxUI
Alexander Potochkin
alexander.potochkin at oracle.com
Mon Jun 3 09:05:55 UTC 2013
Hello Anton
On 6/2/2013 1:56 PM, Anton Litvinov wrote:
> Hello Alexander,
>
> Thank you for the review of this fix. Could you please review a new
> version of the fix, which was changed according to your remarks. The
> file name of the regression test was changed to "bug6337518.java" and
> the test was moved to "test/javax/swing/JComboBox" directory.
>
> Webrev: http://cr.openjdk.java.net/~alitvinov/6337518/webrev.01
Thanks much for changing the test!
Looks great now, approved
alexp
>
> Thank you,
> Anton
>
> On 5/31/2013 9:14 PM, Alexander Potochkin wrote:
>> Hello Anton
>>
>> The fix and the test look good,
>> only a minor comment
>>
>> It is a common practice to name the new Swing tests like this:
>> /ComponenetName/bugNumber/bugNumber.java
>>
>> I think it makes sense to follow the known pattern,
>> so could you rename**NPEOnInstallComponentsTest folder to "6337518"
>>
>> by the way, you can safely put the new test under the JComboBox folder
>> to avoid creating the new one (but this is up to you)
>>
>> Thanks
>> alexp
>>
>> On 5/31/2013 8:03 PM, Anton Litvinov wrote:
>>> Hello,
>>>
>>> Please review the following fix for a bug.
>>>
>>> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6337518
>>> Webrev: http://cr.openjdk.java.net/~alitvinov/6337518/webrev.00
>>>
>>> This bug consists in throwing of NullPointerException from the
>>> method "javax.swing.plaf.basic.BasicComboBoxUI.installComponents()",
>>> if "createArrowButton()" method returns null. The solution moves
>>> code generating this exception into a body of an already existing
>>> "if" block which checks the result of a call to
>>> "createArrowButton()" method for null.
>>>
>>> Thank you,
>>> Anton
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20130603/45c78cb6/attachment.html>
More information about the swing-dev
mailing list