Subject: bug#18371: closed (Re: bug#18371: desktop-read: do not (desktop-clear) when no .emacs.desktop file is found )

From: [email-masked] (GNU bug Tracking System)
Date: Fri, 23 Jan 2015 15:30:05 +0000

Your bug report

#18371: desktop-read: do not (desktop-clear) when no .emacs.desktop file is found 

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
18371: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=18371
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#18371: desktop-read: do not (desktop-clear) when no .emacs.desktop file is found
From: Ivan Shmakov <[email-masked]>
To: [email-masked]
Date: Fri, 23 Jan 2015 15:29:38 +0000
List-Id: <debbugs-submit.debbugs.gnu.org>

Version: 25.1

>>>>> Ivan Shmakov <[email-masked]> writes:

[…]

 > Given that desktop-read does /not/ call desktop-clear in the
 > “.emacs.desktop is found” branch, and that in that case, desktop-read
 > /appends/ to the current session (rather than /replacing/ it), I see
 > no good reason to call desktop-clear from desktop-read in the other
 > case, either.

 > Please thus consider the (trivial) patch MIMEd.

	Pushed; closing.

commit e391d88705efd3897996ed81e1f37b936dc3aa21
CommitDate: 2015-01-17 19:35:22 +0000

    Do not clear the session when trying to read a non-existent desktop file.

    * lisp/desktop.el (desktop-read): Do not call desktop-clear when no
    desktop file is found.

    Fixes: debbugs:18371

-- 
FSF associate member #7257  np. Liar — Motorhead        … 3013 B6A0 230E 334A

Subject: desktop-read: do not (desktop-clear) when no .emacs.desktop file is found
From: Ivan Shmakov <[email-masked]>
To: [email-masked]
Date: Sun, 31 Aug 2014 20:05:17 +0000
List-Id: <debbugs-submit.debbugs.gnu.org>

Package:  emacs
Tags: patch

	As currently implemented, desktop-read calls desktop-clear when
	no .emacs.desktop file is found:

  1114	    (if (file-exists-p (desktop-full-file-name))
⋯
  1192	      ;; No desktop file found.
  1193	      (desktop-clear)
  1194	      (let ((default-directory desktop-dirname))
  1195	        (run-hooks 'desktop-no-desktop-file-hook))
  1196	      (message "No desktop file.")

	While this is likely to be a no-op when desktop-read is called
	from user’s ~/.emacs, – this may actually affect user’s session
	if desktop-read gets called from within an established session.

	And even more so a session containing any “process interaction”
	buffers, – like those one gets with M-x run-scheme or M-x erc.
	(Unless desktop-clear-preserve-buffers is suitably customized
	beforehand, that is.  However, as it seems, that may easily spin
	off the limits of convenience, which may be a bug by itself.)

	Given that desktop-read does /not/ call desktop-clear in the
	“.emacs.desktop is found” branch, and that in that case,
	desktop-read /appends/ to the current session (rather than
	/replacing/ it), I see no good reason to call desktop-clear from
	desktop-read in the other case, either.

	Please thus consider the (trivial) patch MIMEd.

-- 
FSF associate member #7257  http://boycottsystemd.org/  … 3013 B6A0 230E 334A
--- a/lisp/desktop.el
+++ b/lisp/desktop.el
@@ -1190,7 +1190,6 @@
 	    (desktop-auto-save-enable)
 	    t))
       ;; No desktop file found.
-      (desktop-clear)
       (let ((default-directory desktop-dirname))
         (run-hooks 'desktop-no-desktop-file-hook))
       (message "No desktop file.")