changeset 98112:97094e1516ad

(read_key_sequence): Reapply translation maps when switching keyboards.
author Chong Yidong <cyd@stupidchicken.com>
date Tue, 09 Sep 2008 18:16:48 +0000
parents 0471db1b87ac
children 95e740c93248
files src/keyboard.c
diffstat 1 files changed, 6 insertions(+), 12 deletions(-) [+]
line wrap: on
line diff
--- a/src/keyboard.c	Tue Sep 09 18:15:41 2008 +0000
+++ b/src/keyboard.c	Tue Sep 09 18:16:48 2008 +0000
@@ -9154,16 +9154,10 @@
   orig_keymap = get_local_map (PT, current_buffer, Qkeymap);
   from_string = Qnil;
 
-  /* The multi-tty merge moved the code below to right after
-   `replay_sequence' which caused all these translation maps to be applied
-   repeatedly, even tho their doc says very clearly they are not applied to
-   their own output.
-   The reason for this move was: "We may switch keyboards between rescans,
-   so we need to reinitialize fkey and keytran before each replay".
-   This move was wrong (even if we switch keyboards, keybuf still holds the
-   keys we've read already from the original keyboard and some of those keys
-   may have already been translated).  So there may still be a bug out there
-   lurking.  */
+  /* We jump here when we need to reinitialize fkey and keytran; this
+     happens if we switch keyboards between rescans.  */
+ replay_entire_sequence:
+
   indec.map = indec.parent = current_kboard->Vinput_decode_map;
   fkey.map = fkey.parent = current_kboard->Vlocal_function_key_map;
   keytran.map = keytran.parent = Vkey_translation_map;
@@ -9360,7 +9354,7 @@
 		    /* Don't touch interrupted_kboard when it's been
 		       deleted. */
 		    delayed_switch_frame = Qnil;
-		    goto replay_sequence;
+		    goto replay_entire_sequence;
 		  }
 
 		if (!NILP (delayed_switch_frame))
@@ -9392,7 +9386,7 @@
 		mock_input = 0;
 		orig_local_map = get_local_map (PT, current_buffer, Qlocal_map);
 		orig_keymap = get_local_map (PT, current_buffer, Qkeymap);
-		goto replay_sequence;
+		goto replay_entire_sequence;
 	      }
 	  }