<Swing Dev> [8] Review request for JDK-7132383: [macosx] bug6596966.java should be adapted for Mac

vera akulova vera.akulova at oracle.com
Thu Feb 7 15:15:21 UTC 2013


On 07.02.2013 18:41, Alexander Scherbatiy wrote:
>
> Just one typo:
>
>  241         if ((modifiers&  InputEvent.META_MASK) != 0) {
>  242             result.add(KeyEvent.META_MASK);
>  243         }
>
>    It seems that the  KeyEvent.VK_META should be added to the result.
yes, I apologize, next version of the fix is here: 
http://cr.openjdk.java.net/~kshefov/7132383/webrev.02

Thanks,
Vera
>
>   Thanks,
>   Alexandr.
>
> On 2/7/2013 6:32 PM, vera akulova wrote:
>> Hi Alexander,
>>
>> thanks for your reply,
>> please review the new version of fix: 
>> http://cr.openjdk.java.net/~kshefov/7132383/webrev.02/
>>
>> Vera
>>
>> On 07.02.2013 17:18, Alexander Scherbatiy wrote:
>>> On 2/6/2013 3:15 PM, vera akulova wrote:
>>>> Hello,
>>>>
>>>> as we discussed offline it is better to use 
>>>> SwingUtilities2.getSystemMnemonicKeyMask(), I suggest the following 
>>>> fix:
>>>>
>>>> http://cr.openjdk.java.net/~kshefov/7132383/webrev.01/
>>>>
>>>> Could you please review?
>>>
>>>    The fix looks good for me.
>>>
>>>    I would suggest to rename the getKeyCodesByKeyMask method to 
>>> getKeyCodesFromKeyMask and
>>>    add getSystemMnemonicKeyCodes() method to Util class because this 
>>> is often used in the tests.
>>>
>>>    Thanks,
>>>    Alexandr.
>>>
>>>>
>>>> Thanks,
>>>> Vera
>>>>
>>>> On 11.12.2012 18:44, vera akulova wrote:
>>>>> Hello,
>>>>>
>>>>> Please review a fix for the issue:
>>>>>    JDK-7132383: [macosx] bug6596966.java should be adapted for Mac
>>>>>
>>>>> The webrev is http://cr.openjdk.java.net/~kshefov/7132383/webrev.00/
>>>>>
>>>>> Ctrl-Alt-Mnemonic is used now for Mac instead of Alt-Mnemonic.
>>>>> Fixed test works fine on windows, linux, solaris and macos.
>>>>>
>>>>> Thanks,
>>>>> Vera.
>>>
>



More information about the swing-dev mailing list