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

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Thu Feb 7 14:41:12 UTC 2013


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.

   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