(add-change-log-entry): Check add-log-full-name

and add-log-mailing-address later, after change-log-mode-hook.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 6fd0a3b3e44a8faf1ad1bcc5afbc02ae8cb08a09..fc755882a6b00e198ba9f8dd8b7e6099cf1d611c 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2007-07-15  Roland McGrath  <roland@frob.com>
+
+       * add-log.el (add-change-log-entry): Check add-log-full-name
+       and add-log-mailing-address later, after change-log-mode-hook.
+
 2007-07-15  Richard Stallman  <rms@gnu.org>
 
        * isearch.el (isearch-mode, isearch-done):

diff --git a/lisp/add-log.el b/lisp/add-log.el
index 08ce78d371de1a3c682afdf3e7ad336742ebede0..09368d08f9efb36c627d8e3f0a4a27c98654f736 100644 (file)
--- a/lisp/add-log.el
+++ b/lisp/add-log.el
@@ -509,19 +509,19 @@ non-nil, otherwise in local time."
         (file-name (expand-file-name (find-change-log file-name buffer-file)))
         ;; Set ITEM to the file name to use in the new item.
         (item (add-log-file-name buffer-file file-name))
-        bound
-        (full-name (or add-log-full-name (user-full-name)))
-        (mailing-address (or add-log-mailing-address user-mail-address)))
+        bound full-name mailing-address)
 
     (if whoami
        (progn
-         (setq full-name (read-string "Full name: " full-name))
+         (setq full-name (read-string "Full name: "
+                                      (or add-log-full-name (user-full-name))))
          ;; Note that some sites have room and phone number fields in
          ;; full name which look silly when inserted.  Rather than do
          ;; anything about that here, let user give prefix argument so that
          ;; s/he can edit the full name field in prompter if s/he wants.
          (setq mailing-address
-               (read-string "Mailing address: " mailing-address))))
+               (read-string "Mailing address: "
+                            (or add-log-mailing-address user-mail-address)))))
 
     (unless (equal file-name buffer-file-name)
       (if (or other-window (window-dedicated-p (selected-window)))
@@ -532,6 +532,11 @@ non-nil, otherwise in local time."
     (undo-boundary)
     (goto-char (point-min))
 
+    (or full-name
+       (setq full-name (or add-log-full-name (user-full-name))))
+    (or mailing-address
+        (setq mailing-address (or add-log-mailing-address user-mail-address)))
+
     ;; If file starts with a copyright and permission notice, skip them.
     ;; Assume they end at first blank line.
     (when (looking-at "Copyright")