diff options
author | Corinna Vinschen <corinna@vinschen.de> | 2017-01-19 20:59:48 +0300 |
---|---|---|
committer | Corinna Vinschen <corinna@vinschen.de> | 2017-01-19 20:59:48 +0300 |
commit | 4652cc4384c3f24db901010adcad021c19b2b7a2 (patch) | |
tree | cb13eeef38c86190d42e34272589064a453e967d | |
parent | 96267313e1ec5cdd85a4dc3a30a43fce8d8824a2 (diff) |
Handle Alt+Numpad key sequences in console input and select
{p}select/{p}poll completely ignored Alt+Numpad key sequences in console
input which results in newer readline using pselect to fail handling such
sequences correctly. See https://cygwin.com/ml/cygwin/2017-01/msg00135.html
During debugging and testing it turned out that while reading console
input, single key presses during an Alt+Numpad sequences where not
ignored, so ultimately a sequence like
Alt-down Numpad-1 Numpad-2 Numpad-3
whihc is supposed to result in a single character in the input stream
will actually result in 4 chars in the input stream, three control
sequences and the actual character.
Both problems should be fixed by this patch.
Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
-rw-r--r-- | winsup/cygwin/fhandler_console.cc | 12 | ||||
-rw-r--r-- | winsup/cygwin/select.cc | 26 |
2 files changed, 35 insertions, 3 deletions
diff --git a/winsup/cygwin/fhandler_console.cc b/winsup/cygwin/fhandler_console.cc index 1162bfcad..ced4f4b35 100644 --- a/winsup/cygwin/fhandler_console.cc +++ b/winsup/cygwin/fhandler_console.cc @@ -7,6 +7,7 @@ Cygwin license. Please consult the file "CYGWIN_LICENSE" for details. */ #include "winsup.h" +#include <dinput.h> #include "miscfuncs.h" #include <stdio.h> #include <stdlib.h> @@ -415,6 +416,17 @@ fhandler_console::read (void *pv, size_t& buflen) // left alt -- see http://www.microsoft.com/hwdev/tech/input/Scancode.asp && input_rec.Event.KeyEvent.wVirtualScanCode == 0x38)) continue; + /* Ignore Alt+Numpad keys. These are used to enter codepoints not + available in the current keyboard layout. They are eventually + handled in the key-up case below. For details see + http://www.fileformat.info/tip/microsoft/enter_unicode.htm */ + if (input_rec.Event.KeyEvent.bKeyDown + && wch == 0 + && input_rec.Event.KeyEvent.dwControlKeyState == LEFT_ALT_PRESSED + && input_rec.Event.KeyEvent.wVirtualScanCode >= DIK_NUMPAD7 + && input_rec.Event.KeyEvent.wVirtualScanCode <= DIK_NUMPAD0 + && input_rec.Event.KeyEvent.wVirtualScanCode != DIK_SUBTRACT) + continue; if (control_key_state & SHIFT_PRESSED) con.nModifiers |= 1; diff --git a/winsup/cygwin/select.cc b/winsup/cygwin/select.cc index 578973741..1195951cb 100644 --- a/winsup/cygwin/select.cc +++ b/winsup/cygwin/select.cc @@ -12,6 +12,7 @@ details. */ #define __INSIDE_CYGWIN_NET__ #include "winsup.h" +#include <dinput.h> #include <stdlib.h> #include <sys/param.h> #include "ntdll.h" @@ -893,9 +894,28 @@ peek_console (select_record *me, bool) fh->send_winch_maybe (); if (irec.EventType == KEY_EVENT) { - if (irec.Event.KeyEvent.bKeyDown - && (irec.Event.KeyEvent.uChar.AsciiChar - || fhandler_console::get_nonascii_key (irec, tmpbuf))) + if (irec.Event.KeyEvent.bKeyDown) + { + /* Ignore Alt+Numpad keys. These are used to enter codepoints + not available in the current keyboard layout. They are + eventually handled in the key-up case below. For details see + http://www.fileformat.info/tip/microsoft/enter_unicode.htm */ + if (irec.Event.KeyEvent.uChar.UnicodeChar == 0 + && irec.Event.KeyEvent.dwControlKeyState == LEFT_ALT_PRESSED + && irec.Event.KeyEvent.wVirtualScanCode >= DIK_NUMPAD7 + && irec.Event.KeyEvent.wVirtualScanCode <= DIK_NUMPAD0 + && irec.Event.KeyEvent.wVirtualScanCode != DIK_SUBTRACT) + ; + /* Handle normal input. */ + else if (irec.Event.KeyEvent.uChar.UnicodeChar + || fhandler_console::get_nonascii_key (irec, tmpbuf)) + return me->read_ready = true; + } + /* Ignore key up events, except for left alt events with + non-zero character */ + else if (irec.Event.KeyEvent.uChar.UnicodeChar != 0 + && irec.Event.KeyEvent.wVirtualKeyCode == VK_MENU + && irec.Event.KeyEvent.wVirtualScanCode == 0x38) return me->read_ready = true; } else |