* xterm.c (syms_of_xterm): staticpro Qalt, Qhyper, Qmeta, and Qsuper.

This is safer, and it's what w32fns.c does.

diff --git a/src/ChangeLog b/src/ChangeLog
index 7ea7fb84ec9f10dd940c2d43c812dfc1af105e99..08bbe9e69d238f0fe0c56bf1a8634606cb8d6d31 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,6 +1,7 @@
 2013-11-12  Paul Eggert  <eggert@cs.ucla.edu>
 
-       * xterm.c (syms_of_xterm): staticpro Qmodifier_value.
+       * xterm.c (syms_of_xterm): staticpro Qmodifier_value, Qalt, Qhyper,
+       Qmeta, and Qsuper.  This is safer, and it's what w32fns.c does.
 
        * buffer.c (Fforce_mode_line_update): Don't fall off end of function
        that requires a return value.

diff --git a/src/xterm.c b/src/xterm.c
index b803cc13258698ceb629c4915c653ce6e3724f87..446b2cf1e45171182fda306b07dcabd0d5ae2977 100644 (file)
--- a/src/xterm.c
+++ b/src/xterm.c
@@ -10588,13 +10588,13 @@ With MS Windows or Nextstep, the value is t.  */);
 #endif
 
   DEFSYM (Qmodifier_value, "modifier-value");
-  Qalt = intern_c_string ("alt");
+  DEFSYM (Qalt, "alt");
   Fput (Qalt, Qmodifier_value, make_number (alt_modifier));
-  Qhyper = intern_c_string ("hyper");
+  DEFSYM (Qhyper, "hyper");
   Fput (Qhyper, Qmodifier_value, make_number (hyper_modifier));
-  Qmeta = intern_c_string ("meta");
+  DEFSYM (Qmeta, "meta");
   Fput (Qmeta, Qmodifier_value, make_number (meta_modifier));
-  Qsuper = intern_c_string ("super");
+  DEFSYM (Qsuper, "super");
   Fput (Qsuper, Qmodifier_value, make_number (super_modifier));
 
   DEFVAR_LISP ("x-alt-keysym", Vx_alt_keysym,