From: simon <simon@cda61777-01e9-0310-a592-d414129be87e>
Date: Sat, 15 Dec 2001 11:49:48 +0000 (+0000)
Subject: Reverse part of RDB's pedantic patch from rev 1.129 [r1105]: app
X-Git-Url: https://git.distorted.org.uk/~mdw/sgt/putty/commitdiff_plain/3a9531212a6d0077e566912d4790020315d15eb1?hp=a3cfd0c8f4e800fe7f5d5789e9bfd29b7dd75c54

Reverse part of RDB's pedantic patch from rev 1.129 [r1105]: app
cursor and app keypad modes are now once again independently
switchable. The VT100 and VT102 manuals may have done it RDB's way
as he said, but xterm and rxvt disagree and at least one app depends
on the xterm way. It'll have to become a configurable option if
anyone has trouble with this.


git-svn-id: svn://svn.tartarus.org/sgt/putty@1495 cda61777-01e9-0310-a592-d414129be87e
---

diff --git a/window.c b/window.c
index a8fe7ab5..1b61b6c6 100644
--- a/window.c
+++ b/window.c
@@ -3504,11 +3504,24 @@ static int TranslateKey(UINT message, WPARAM wParam, LPARAM lParam,
 		    p += sprintf((char *) p, "\x1B%c", xkey);
 		else {
 		    int app_flg = (app_cursor_keys && !cfg.no_applic_c);
-		    /* VT100 & VT102 manuals both state the app cursor keys
-		     * only work if the app keypad is on.
+#if 0
+		    /*
+		     * RDB: VT100 & VT102 manuals both state the
+		     * app cursor keys only work if the app keypad
+		     * is on.
+		     * 
+		     * SGT: That may well be true, but xterm
+		     * disagrees and so does at least one
+		     * application, so I've #if'ed this out and the
+		     * behaviour is back to PuTTY's original: app
+		     * cursor and app keypad are independently
+		     * switchable modes. If anyone complains about
+		     * _this_ I'll have to put in a configurable
+		     * option.
 		     */
 		    if (!app_keypad_keys)
 			app_flg = 0;
+#endif
 		    /* Useful mapping of Ctrl-arrows */
 		    if (shift_state == 2)
 			app_flg = !app_flg;