[Emacs-diffs] /srv/bzr/emacs/trunk r99749: Close bug#5755.

[Glenn Morris]

------------------------------------------------------------
revno: 99749
committer: Glenn Morris <address@hidden>
branch nick: trunk
timestamp: Wed 2010-03-24 23:18:17 -0700
message:
  Close bug#5755.
  
  * desktop.el (desktop-save-buffer-p): Don't mistakenly include
  all dired buffers, even tramp ones.
modified:
  lisp/ChangeLog
  lisp/desktop.el

=== modified file 'lisp/ChangeLog'
--- a/lisp/ChangeLog    2010-03-25 00:06:08 +0000
+++ b/lisp/ChangeLog    2010-03-25 06:18:17 +0000
@@ -1,3 +1,8 @@
+2010-03-25  Glenn Morris  <address@hidden>
+
+       * desktop.el (desktop-save-buffer-p): Don't mistakenly include
+       all dired buffers, even tramp ones.  (Bug#5755)
+
 2010-03-25  Stefan Monnier  <address@hidden>
 
        Add "union tags" in mpc.el.

=== modified file 'lisp/desktop.el'
--- a/lisp/desktop.el   2010-01-13 08:35:10 +0000
+++ b/lisp/desktop.el   2010-03-25 06:18:17 +0000
@@ -1,7 +1,8 @@
 ;;; desktop.el --- save partial status of Emacs when killed
 
 ;; Copyright (C) 1993, 1994, 1995, 1997, 2000, 2001, 2002, 2003,
-;;   2004, 2005, 2006, 2007, 2008, 2009, 2010 Free Software Foundation, Inc.
+;;   2004, 2005, 2006, 2007, 2008, 2009, 2010
+;;   Free Software Foundation, Inc.
 
 ;; Author: Morten Welinder <address@hidden>
 ;; Keywords: convenience
@@ -811,19 +812,23 @@
 FILENAME is the visited file name, BUFNAME is the buffer name, and
 MODE is the major mode.
 \n\(fn FILENAME BUFNAME MODE)"
-  (let ((case-fold-search nil))
+  (let ((case-fold-search nil)
+        dired-skip)
     (and (not (and (stringp desktop-buffers-not-to-save)
                   (not filename)
                   (string-match desktop-buffers-not-to-save bufname)))
          (not (memq mode desktop-modes-not-to-save))
+         ;; FIXME this is broken if desktop-files-not-to-save is nil.
          (or (and filename
                  (stringp desktop-files-not-to-save)
                   (not (string-match desktop-files-not-to-save filename)))
              (and (eq mode 'dired-mode)
                   (with-current-buffer bufname
-                    (not (string-match desktop-files-not-to-save
-                                       default-directory))))
+                    (not (setq dired-skip
+                               (string-match desktop-files-not-to-save
+                                             default-directory)))))
              (and (null filename)
+                  (null dired-skip)     ; bug#5755
                  (with-current-buffer bufname desktop-save-buffer))))))
 
 ;; ----------------------------------------------------------------------------