changeset 51014:7dde1bfbb922

(apply_modifiers): Don't fill the other cache.
author Stefan Monnier <monnier@iro.umontreal.ca>
date Thu, 15 May 2003 21:20:52 +0000
parents db5c6e7f697f
children e59a4e907ba0
files src/keyboard.c
diffstat 1 files changed, 9 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/src/keyboard.c	Thu May 15 15:36:22 2003 +0000
+++ b/src/keyboard.c	Thu May 15 21:20:52 2003 +0000
@@ -6091,12 +6091,15 @@
       entry = Fcons (index, new_symbol);
       Fput (base, Qmodifier_cache, Fcons (entry, cache));
 
-      /* We have the parsing info now for free, so add it to the caches.  */
-      XSETFASTINT (index, modifiers);
-      Fput (new_symbol, Qevent_symbol_element_mask,
-	    Fcons (base, Fcons (index, Qnil)));
-      Fput (new_symbol, Qevent_symbol_elements,
-	    Fcons (base, lispy_modifier_list (modifiers)));
+      /* We have the parsing info now for free, so we could add it to
+	 the caches:
+         XSETFASTINT (index, modifiers);
+         Fput (new_symbol, Qevent_symbol_element_mask,
+               Fcons (base, Fcons (index, Qnil)));
+         Fput (new_symbol, Qevent_symbol_elements,
+               Fcons (base, lispy_modifier_list (modifiers)));
+	 Sadly, this is only correct if `base' is indeed a base event,
+	 which is not necessarily the case.  -stef  */
     }
 
   /* Make sure this symbol is of the same kind as BASE.