* lisp/saveplace.el (toggle-save-place, save-place-to-alist)

(save-places-to-alist, save-place-dired-hook): Add (derived-mode-p
'dired-mode) before checking for dired-directory.

Fixes: debbugs:16477

2 files changed, 22 insertions, 10 deletions

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 2f76b7502a3..3a838762a1b 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,5 +1,11 @@
 2014-01-20  Juri Linkov  <juri@jurta.org>
 
+        * saveplace.el (toggle-save-place, save-place-to-alist)
+        (save-places-to-alist, save-place-dired-hook): Add (derived-mode-p
+        'dired-mode) before checking for dired-directory.  (Bug#16477)
+
+2014-01-20  Juri Linkov  <juri@jurta.org>
+
         * indent.el (indent-line-to): Use backward-to-indentation
         instead of back-to-indentation.  (Bug#16461)
 
diff --git a/lisp/saveplace.el b/lisp/saveplace.el
index f77a438473f..064a70a1041 100644
--- a/lisp/saveplace.el
+++ b/lisp/saveplace.el
@@ -152,7 +152,8 @@ file:
 
 \(setq-default save-place t\)"
   (interactive "P")
-  (if (not (or buffer-file-name dired-directory))
+  (if (not (or buffer-file-name (and (derived-mode-p 'dired-mode)
+                                     dired-directory)))
       (message "Buffer `%s' not visiting a file or directory" (buffer-name))
     (if (and save-place (or (not parg) (<= parg 0)))
         (progn
@@ -172,10 +173,11 @@ file:
   ;; will be saved again when Emacs is killed.
   (or save-place-loaded (load-save-place-alist-from-file))
   (let ((item (or buffer-file-name
-                  (and dired-directory
-                       (if (consp dired-directory)
-                           (expand-file-name (car dired-directory))
-                         (expand-file-name dired-directory))))))
+                  (and (derived-mode-p 'dired-mode)
+                       dired-directory
+                       (expand-file-name (if (consp dired-directory)
+                                             (car dired-directory)
+                                           dired-directory))))))
     (when (and item
                (or (not save-place-ignore-files-regexp)
                    (not (string-match save-place-ignore-files-regexp
@@ -184,7 +186,8 @@ file:
             (position (cond ((eq major-mode 'hexl-mode)
                              (with-no-warnings
                                (1+ (hexl-current-address))))
-                            (dired-directory
+                            ((and (derived-mode-p 'dired-mode)
+                                  dired-directory)
                              (let ((filename (dired-get-filename nil t)))
                                (if filename
                                    `((dired-filename . ,filename))
@@ -301,7 +304,8 @@ may have changed\) back to `save-place-alist'."
       (with-current-buffer (car buf-list)
         ;; save-place checks buffer-file-name too, but we can avoid
         ;; overhead of function call by checking here too.
-        (and (or buffer-file-name dired-directory)
+        (and (or buffer-file-name (and (derived-mode-p 'dired-mode)
+                                       dired-directory))
              (save-place-to-alist))
         (setq buf-list (cdr buf-list))))))
 
@@ -321,9 +325,11 @@ may have changed\) back to `save-place-alist'."
 (defun save-place-dired-hook ()
   "Position the point in a dired buffer."
   (or save-place-loaded (load-save-place-alist-from-file))
-  (let ((cell (assoc (if (consp dired-directory)
-                         (expand-file-name (car dired-directory))
-                       (expand-file-name dired-directory))
+  (let ((cell (assoc (and (derived-mode-p 'dired-mode)
+                          dired-directory
+                          (expand-file-name (if (consp dired-directory)
+                                                (car dired-directory)
+                                              dired-directory)))
                      save-place-alist)))
     (if cell
         (progn