* xdisp.c (select_frame_for_redisplay): Adjust for last change to

indirect_variable.
* eval.c (lisp_indirect_variable): New fun.
(Fuser_variable_p): Use it.

diff --git a/src/ChangeLog b/src/ChangeLog
index 0232ce4a1ca9e8491a916852a0bd8835c4fc1b11..9eb5f95b5fb4a7c1703b0732af0774d73a223bce 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,10 @@
+2008-05-20  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * xdisp.c (select_frame_for_redisplay): Adjust for last change to
+       indirect_variable.
+       * eval.c (lisp_indirect_variable): New fun.
+       (Fuser_variable_p): Use it.
+
 2008-05-19  Stefan Monnier  <monnier@iro.umontreal.ca>
 
        * lisp.h (indirect_variable):

diff --git a/src/eval.c b/src/eval.c
index 2652064e064176ede91745a0fed4644a3c34a16d..076ac91cb6f885c99bcd045e2059c579a5f67fed 100644 (file)
--- a/src/eval.c
+++ b/src/eval.c
@@ -934,6 +934,13 @@ user_variable_p_eh (ignore)
   return Qnil;
 }
 
+static Lisp_Object
+lisp_indirect_variable (Lisp_Object sym)
+{
+  XSETSYMBOL (sym, indirect_variable (XSYMBOL (sym)));
+  return sym;
+}
+
 DEFUN ("user-variable-p", Fuser_variable_p, Suser_variable_p, 1, 1, 0,
        doc: /* Return t if VARIABLE is intended to be set and modified by users.
 \(The alternative is a variable used internally in a Lisp program.)
@@ -954,7 +961,7 @@ chain of symbols.  */)
 
   /* If indirect and there's an alias loop, don't check anything else.  */
   if (XSYMBOL (variable)->indirect_variable
-      && NILP (internal_condition_case_1 (indirect_variable, variable,
+      && NILP (internal_condition_case_1 (lisp_indirect_variable, variable,
                                           Qt, user_variable_p_eh)))
     return Qnil;
 

diff --git a/src/xdisp.c b/src/xdisp.c
index 9f0c173d5b7d8175c2c1acffae2fbfc800b6f138..83e84861425e0e91ccf0ad9eb71bf448df24d388 100644 (file)
--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -11033,8 +11033,9 @@ static void
 select_frame_for_redisplay (frame)
      Lisp_Object frame;
 {
-  Lisp_Object tail, sym, val;
+  Lisp_Object tail, symbol, val;
   Lisp_Object old = selected_frame;
+  struct Lisp_Symbol *sym;
 
   xassert (FRAMEP (frame) && FRAME_LIVE_P (XFRAME (frame)));
 
@@ -11044,15 +11045,15 @@ select_frame_for_redisplay (frame)
     {
       for (tail = XFRAME (frame)->param_alist; CONSP (tail); tail = XCDR (tail))
        if (CONSP (XCAR (tail))
-           && (sym = XCAR (XCAR (tail)),
-               SYMBOLP (sym))
-           && (sym = indirect_variable (sym),
-               val = SYMBOL_VALUE (sym),
+           && (symbol = XCAR (XCAR (tail)),
+               SYMBOLP (symbol))
+           && (sym = indirect_variable (XSYMBOL (symbol)),
+               val = sym->value,
                (BUFFER_LOCAL_VALUEP (val)))
            && XBUFFER_LOCAL_VALUE (val)->check_frame)
          /* Use find_symbol_value rather than Fsymbol_value
             to avoid an error if it is void.  */
-         find_symbol_value (sym);
+         find_symbol_value (symbol);
     } while (!EQ (frame, old) && (frame = old, 1));
 }