GNU bug report logs - #28843
26.0.90; gnus kills unsaved message buffer

Previous Next

Packages: emacs, gnus;

Reported by: Nick Helm <nick <at> tenpoint.co.nz>

Date: Sun, 15 Oct 2017 07:47:02 UTC

Severity: normal

Tags: fixed

Found in version 26.0.90

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Nick Helm <nick <at> tenpoint.co.nz>
To: 28843 <at> debbugs.gnu.org
Subject: bug#28843: 26.0.90; gnus kills unsaved message buffer
Date: Wed, 08 Nov 2017 15:28:50 +1300
On Thu, 26 Oct 2017 at 13:52:13 +1300, Nick Helm wrote:

> On Sun, 15 Oct 2017 at 20:46:25 +1300, Nick Helm wrote:
>
>> Gnus exits, and the unsaved message buffer dies with it, without prompts
>> to save.
>
> It seems the behaviour is intentional ... This commit changed
> `gnus-clear-system' to include this:
>
>   #+begin_src emacs-lisp
>   ;; Kill Gnus buffers.
>   (do-auto-save t)
>   (dolist (buffer (gnus-buffers))
>     (when (gnus-buffer-exists-p buffer)
>       (with-current-buffer buffer
>   	   (set-buffer-modified-p nil)
>   	   (when (local-variable-p 'kill-buffer-hook)
>   	     (setq kill-buffer-hook nil))))
>     (gnus-kill-buffer buffer))
>   #+end_src
>
> So gnus is at least auto-saving draft messages before zapping them.
>
> Is there a better way to do this though? I think the user should at
> least have some warning that an unsaved buffer is about to be
> automatically killed.

One solution (though not a very good one IMHO) would be to make the
auto-save depend on the user's value of guns-interactive-exit. For
example:

--- a/lisp/gnus/gnus-start.el	2017-10-26 12:49:43.000000000 +1300
+++ b/lisp/gnus/gnus-start.el	2017-10-26 12:45:12.000000000 +1300
@@ -731,11 +731,12 @@
     (kill-buffer (get-file-buffer (gnus-newsgroup-kill-file nil))))
   (gnus-kill-buffer nntp-server-buffer)
   ;; Kill Gnus buffers.
-  (do-auto-save t)
   (dolist (buffer (gnus-buffers))
     (when (gnus-buffer-exists-p buffer)
       (with-current-buffer buffer
-	(set-buffer-modified-p nil)
+        (unless gnus-interactive-exit
+          (do-auto-save t t)
+	(set-buffer-modified-p nil))
 	(when (local-variable-p 'kill-buffer-hook)
 	  (setq kill-buffer-hook nil))))
     (gnus-kill-buffer buffer))






This bug report was last modified 7 years and 40 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.