.hgtags
author Sam Lantinga <slouken@libsdl.org>
Sat, 27 Jul 2013 03:20:09 -0700
changeset 7526 6c0f0280149c
parent 4462 5cb6424a5676
child 7578 4b43278168be
permissions -rw-r--r--
Fixed bug 1272 - Bogus numlock key up/down events being reported on MacOS X Vern Jensen The problem is that in certain situations I'm getting THREE keyUp/keyDown events when I push certain keys. In my event code I added: case SDL_KEYUP: printf("SDL KeyScanCode for KEYUP event: %d\n", event->key.keysym.scancode ); … and case SDL_KEYDOWN: printf("SDL KeyScanCode for KEYDOWN event: %d\n", event->key.keysym.scancode ); … The result of one test run where I push 2 keys and then release them is this: SDL KeyScanCode for KEYDOWN event: 92 // Pushed keypad 4 SDL KeyScanCode for KEYDOWN event: 83 // Pushed left shift SDL KeyScanCode for KEYUP event: 83 SDL KeyScanCode for KEYDOWN event: 225 SDL KeyScanCode for KEYUP event: 92 // Released keypad 4 SDL KeyScanCode for KEYDOWN event: 83 SDL KeyScanCode for KEYUP event: 83 SDL KeyScanCode for KEYUP event: 225 // Released left shift There *should* be only a total of 4 events above… 2 for each key being pushed, and 2 for each being released. But instead some bogus events for numlock being pushed/released are sent from SDL. These events did not occur. I did not push numlock. The value above for numlock is 83. Comments above show when I pushed each key. As you can see, when I push left shift, THREE events are instantly sent to my application, keyDown and then keyUp for numlock, and then the valid event for left shift (the key that was actually pushed). You could replace keypad 4 with pretty much any keyPad key and it'll still happen. You can also replace it with any arrow key and it'll happen. However, when trying it with normal letter keys on the main keyboard it didn't. It happens with other modifier keys too, not just left shift. The order in which the keys are pressed matter. For instance, if I do: 1) keypad 4 2) left shift 3) release left shift 4) release keypad 4 Then at step 2, I get the 3 events above (when there should be only one), but steps 3 and 4 work properly… I don't get extra keyUp/keyDown events for steps 3 or 4. Thereas if the order of steps 3 and 4 are reversed, I get the bogus extra events for numlock. Also, the problem can occur even when pushing just a single key by itself. If I push left shift, then keypad 4, then release left shift, then release keypad 4, then the following push of left shift will cause the bug. If I continue pushing and releasing left shift though, it won't happen again until I again involve keypad keys. --- Sam Lantinga According to the Apple documentation, NSNumericPadKeyMask is set for any arrow or numeric keypad event. Indeed this is what's happening. I verified that we get the correct events for the numlock key and the mod state gets set correcly, so it should be safe to remove this bogus code.
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
4405
7828bb19af1e update tags
convert-repo
parents:
diff changeset
     1
0afe0e38e02cf2048e93582f01c52fbb91d3c7bb release-1.2.7
7828bb19af1e update tags
convert-repo
parents:
diff changeset
     2
230b156829ed13b31134d96f689c917981f57b84 release-1.2.5
7828bb19af1e update tags
convert-repo
parents:
diff changeset
     3
27cab50ec9c746e886ce0f3fdaa0b0cdc55a594f release-1.2.11
7828bb19af1e update tags
convert-repo
parents:
diff changeset
     4
2fe3fbd2bff50165b3cad33bf40d70b3bb3c9fd0 release-1.2.3
7828bb19af1e update tags
convert-repo
parents:
diff changeset
     5
3c052d3bcc76c899dfd4846be76243a78e8c7180 release-1.2.4
7828bb19af1e update tags
convert-repo
parents:
diff changeset
     6
3c5eed71a3320962551af3b3dfbee0c99fcf0086 release-1.2.10
7828bb19af1e update tags
convert-repo
parents:
diff changeset
     7
4867f7f7dd3426d1dbbeef48b3f3b3aa19590cc4 release-1.2.12
7828bb19af1e update tags
convert-repo
parents:
diff changeset
     8
6e28dae59e3baf4447c83e833a8d2ac912536f5b release-1.2.1
7828bb19af1e update tags
convert-repo
parents:
diff changeset
     9
7c2589fb8d4df54c6faabd3faebd0c0e73f67879 release-1.2.13
7828bb19af1e update tags
convert-repo
parents:
diff changeset
    10
86de11faf082881ad9b73a1a1d78733ca07f8db8 release-1.2.6
7828bb19af1e update tags
convert-repo
parents:
diff changeset
    11
bb051fa871aa0b53ea57df56a446cec3bb85924c release-1.2.2
7828bb19af1e update tags
convert-repo
parents:
diff changeset
    12
cfcb2e1c36ebe9809577adf768b0ec53e8768af9 release-1.2.8
7828bb19af1e update tags
convert-repo
parents:
diff changeset
    13
e044e7c70a50a2f54d14ee20d0933e904e5853b6 release-1.2.9
7828bb19af1e update tags
convert-repo
parents:
diff changeset
    14
f14cf9d71233934811774f941d0de121d5f96ccf release-1.2.14
4462
5cb6424a5676 Tagging version before everything gets ripped out.
Sam Lantinga <slouken@libsdl.org>
parents: 4405
diff changeset
    15
39c22a953456f6c9e2c8993c8ff973824104102a pre-touch-removal