bug#32214: [PATCH] 'message-kill-buffer': fix backup delete question

[Sven Willner]

good idea!

have incorporated that in the appended patch.
also, the 'when' part should only include the actual deletion of 
the backup file
>         (ignore-errors
>           (delete-file auto-save-file-name))
rather than also draft diassociation
>         (let ((message-draft-article draft-article))
>           (message-disassociate-draft)))
also fixed in the updated patch.

Sven

0001-message-kill-buffer-fix-backup-delete-question.patch
Description: Text Data

Robert Pluim <rpluim@gmail.com> writes:

> Sven Willner <sven.willner@gmail.com> writes:
>
> >         (ignore-errors
> >           (delete-file auto-save-file-name))
> >         (let ((message-draft-article draft-article))
> >           (message-disassociate-draft)))
> > + (when (and (and auto-save-file-name
> > + (file-exists-p auto-save-file-name))
> > + (and file-name
> > + (file-exists-p file-name))
>
> (when (and auto-save-file-name
>            (file-exists-p auto-save-file-name)
>            file-name
>            (file-exists-p file-name))
>
> perhaps? You could even drop the 'when'.
>
> Robert