<Swing Dev> [9] Review request for 8049533 SwingUtilities.convertMouseEvent misses MouseWheelEvent.preciseWheelRotation

Petr Pchelko petr.pchelko at oracle.com
Thu Jul 10 16:42:02 UTC 2014


Hello, Alexander.

Still looks good to me.

With best regards. Petr.

> On Jul 10, 2014, at 6:58 PM, Alexander Scherbatiy <alexandr.scherbatiy at oracle.com> wrote:
> 
> 
> Could you review the updated version:
>   http://cr.openjdk.java.net/~alexsch/8049533/webrev.02/
> 
>  The xAbs and yAbs fields are also copied in the LWComponentPeer.createDelegateEvent() and ResizeBox.mouseWheelMoved() methods.
> 
>  Thanks,
>  Alexandr.
> 
> On 7/10/2014 6:45 PM, Petr Pchelko wrote:
>> Hello, Alexander.
>> 
>> The .01 version looks good.
>> 
>> With best regards. Petr.
>> 
>> On 10 июля 2014 г., at 18:32, Alexander Scherbatiy <alexandr.scherbatiy at oracle.com> wrote:
>> 
>>> Could you review the updated fix:
>>>    http://cr.openjdk.java.net/~alexsch/8049533/webrev.01
>>> 
>>> The preciseWheelRotation field is copied in the LWComponentPeer.createDelegateEvent() and ResizeBox.mouseWheelMoved() methods.
>>> 
>>> Thanks,
>>> Alexandr.
>>> 
>>> On 7/10/2014 2:34 PM, Sergey Bylokhov wrote:
>>>> Hi, Alexander.
>>>> Same fix should be applied to LWComponentPeer.createDelegateEvent(), and probably to ResizeBox.mouseWheelMoved. Please check all place where getWheelRotation is used.
>>>> 
>>>> On 10.07.2014 14:21, Alexander Scherbatiy wrote:
>>>>> Hello,
>>>>> 
>>>>> Could you review the fix:
>>>>>  bug: https://bugs.openjdk.java.net/browse/JDK-8049533
>>>>>  webrev: http://cr.openjdk.java.net/~alexsch/8049533/webrev.00
>>>>> 
>>>>> 
>>>>>  The preciseWheelRotation field copying is missed in the SwingUtilities.convertMouseEvent(...) method.
>>>>> 
>>>>> Thanks,
>>>>> Alexandr.
>>>>> 
>>>> 
> 




More information about the swing-dev mailing list