<Swing Dev> Fix for JDK-8065861 : Pressing Esc does not set 'canceled' property of ProgressMonitor
Philip Race
philip.race at oracle.com
Wed Jun 22 16:33:52 UTC 2016
+1
-phil
On 6/22/16, 5:32 AM, Alexandr Scherbatiy wrote:
> The fix looks good to me.
>
> Thanks,
> Alexandr.
>
> On 6/22/2016 3:26 PM, Ajit Ghaisas wrote:
>> Hi,
>>
>> There was a suggestion to modify the Javadoc text.
>> The code changes stand unchanged from webrev.01. Only Javadoc
>> comment has been modified.
>>
>> Request you to review again :
>> http://cr.openjdk.java.net/~aghaisas/8065861/webrev.02/
>>
>> Regards,
>> Ajit
>>
>> -----Original Message-----
>> From: Rajeev Chamyal
>> Sent: Monday, June 13, 2016 1:47 PM
>> To: Alexander Scherbatiy; Ajit Ghaisas; Sergey Bylokhov;
>> swing-dev at openjdk.java.net
>> Subject: RE: Fix for JDK-8065861 : Pressing Esc does not set
>> 'canceled' property of ProgressMonitor
>>
>> Looks good to me.
>>
>> Regards,
>> Rajeev Chamyal
>>
>> -----Original Message-----
>> From: Alexandr Scherbatiy
>> Sent: 10 June 2016 13:06
>> To: Ajit Ghaisas; Sergey Bylokhov; Rajeev Chamyal;
>> swing-dev at openjdk.java.net
>> Subject: Re: Fix for JDK-8065861 : Pressing Esc does not set
>> 'canceled' property of ProgressMonitor
>>
>>
>> The fix looks good to me.
>>
>> Thanks,
>> Alexandr.
>>
>> On 6/10/2016 8:50 AM, Ajit Ghaisas wrote:
>>> Hi,
>>>
>>> Thanks Alex for spotting probable exceptions in code changes.
>>> I have corrected the code to address them.
>>>
>>> Here is the updated webrev. Request you to review.
>>> http://cr.openjdk.java.net/~aghaisas/8065861/webrev.01/
>>>
>>> Regards,
>>> Ajit
>>>
>>> -----Original Message-----
>>> From: Alexandr Scherbatiy
>>> Sent: Thursday, June 09, 2016 8:56 PM
>>> To: Ajit Ghaisas; Sergey Bylokhov; Rajeev Chamyal;
>>> swing-dev at openjdk.java.net
>>> Subject: Re: Fix for JDK-8065861 : Pressing Esc does not set
>>> 'canceled' property of ProgressMonitor
>>>
>>> On 6/8/2016 5:35 PM, Ajit Ghaisas wrote:
>>>> Hi,
>>>>
>>>> Bug :
>>>> https://bugs.openjdk.java.net/browse/JDK-8065861
>>>>
>>>> Issue :
>>>> Pressing Esc does not set 'canceled' property of ProgressMonitor
>>>>
>>>> Analysis :
>>>> ProgressMonitor option pane only gets hidden on pressing Escape key.
>>>> It is not truly canceled as isCanceled() method continues to return
>>>> false.
>>>>
>>>> Fix :
>>>> On Pressing Escape key JOptionPane.CLOSED_OPTION value is set in
>>>> JOptionPane from BasicOptionPaneUI.java.
>>>> This value is used as a condition in isCanceled() to identify
>>>> ProgressMonitor is canceled by pressing Escape key.
>>>>
>>>> Please review the webrev:
>>>> http://cr.openjdk.java.net/~aghaisas/8065861/webrev.00/
>>> The updated code calls methods from 'v' object before it is
>>> checked to the null which can lead to the NPE. The cancelOption[0]
>>> also can throw the IOBE before the arrays length checking.
>>>
>>> Thanks,
>>> Alexandr.
>>>> On review completion, I will raise a CCC request for documentation
>>>> change.
>>>>
>>>> Regards,
>>>> Ajit
>
More information about the swing-dev
mailing list