GNU bug report logs - #74881
31.0.50; [PATCH] * lisp/vc/ediff-util.el: Remove temp files on kill-emacs

Previous Next

Full log

Message #43 received at 74881 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Michael Heerdegen <michael_heerdegen <at> web.de>
Cc: sunlin7 <at> hotmail.com, 74881 <at> debbugs.gnu.org
Subject: Re: bug#74881: 31.0.50; [PATCH] * lisp/vc/ediff-util.el: Remove
 temp files on kill-emacs
Date: Sun, 29 Dec 2024 09:04:47 +0200

> From: Michael Heerdegen <michael_heerdegen <at> web.de>
> Cc: Eli Zaretskii <eliz <at> gnu.org>,  "74881 <at> debbugs.gnu.org"
>  <74881 <at> debbugs.gnu.org>
> Date: Sun, 29 Dec 2024 02:49:14 +0100
> 
> Second: Can we move the `ignore-errors' inwards so that an error in one
> case doesn't abort the complete loop?

Binding inhibit-interaction non-nil could cause the body signal an
error, so this should be taken into account if we move ignore-errors
inwards.  This code must NOT signal any errors, ever.  Which AFAIU
means that if we want ignore-errors not to abort the rest of the loop
(why not, btw? these are just temporary files, after all), we should
redesign this loop such that it catches the errors and continues with
the rest of the buffers.

> >  ;;; Setup functions
> >
> > @@ -488,6 +496,7 @@ ediff-setup
> >        (if (ediff-buffer-live-p ediff-meta-buffer)
> >  	  (ediff-update-meta-buffer
> >  	   ediff-meta-buffer nil ediff-meta-session-number))
> > +      (add-hook 'kill-emacs-hook #'ediff--delete-temp-files-on-kill-emacs)
> 
> Would it be ok to avoid the above loop completely by using the buffer
> local version of `kill-emacs-hook' instead?

You assume that the temporary Ediff files are always visited in some
buffer?  Is that assumption true?

Previous Next

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