(define-obsolete-variable-alias): Use dolist,

and only put a prop if it is non-nil.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 8d211786172bdf2c8c614d37ee30c3812a11bb23..99fc1f1edd00f6ba48d15d54a640d090133ada2c 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2009-11-03  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * emacs-lisp/byte-run.el (define-obsolete-variable-alias): Use dolist,
+       and only put a prop if it is non-nil.
+
 2009-11-03  Juri Linkov  <juri@jurta.org>
 
        * menu-bar.el (menu-bar-make-mm-toggle, menu-bar-make-toggle)

diff --git a/lisp/emacs-lisp/byte-run.el b/lisp/emacs-lisp/byte-run.el
index a00fdf0e4f56de8dcc45778ab0a34afeec8fc5bd..af3150ffe6078c85fc37e334eb50d9fb25a3cc66 100644 (file)
--- a/lisp/emacs-lisp/byte-run.el
+++ b/lisp/emacs-lisp/byte-run.el
@@ -202,10 +202,10 @@ CURRENT-NAME, if it does not already have them:
   `(progn
      (defvaralias ,obsolete-name ,current-name ,docstring)
      ;; See Bug#4706.
-     (mapc (lambda (prop) (or (get ,current-name prop)
-                              (put ,current-name prop
-                                   (get ,obsolete-name prop))))
-           '(saved-value saved-variable-comment))
+     (dolist (prop '(saved-value saved-variable-comment))
+       (and (get ,obsolete-name prop)
+            (null (get ,current-name prop))
+            (put ,current-name prop (get ,obsolete-name prop))))
      (make-obsolete-variable ,obsolete-name ,current-name ,when)))
 (set-advertised-calling-convention
  ;; New code should always provide the `when' argument.