[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#20832: 25.0.50; todo-show accidentally deleted my todo file.
From: |
Stephen Berman |
Subject: |
bug#20832: 25.0.50; todo-show accidentally deleted my todo file. |
Date: |
Fri, 19 Jun 2015 17:14:31 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) |
On Thu, 18 Jun 2015 13:52:21 +0200 Nicolas Richard <nrichard@members.fsf.org>
wrote:
> Le 18/06/2015 12:15, Stephen Berman a écrit :
>> But if there are other
>> triggers, then your patch may be needed. However, since I haven't found
>> any other way to make the problem you encountered occur, I'm inclined to
>> install just the patch below; that way, if there is some other way to
>> get an empty but modified todo-mode buffer, we may find it and be able
>> to fix the underlying cause.
>
> Throwing an error seemed like a good way to find such problems, to me,
> better so than silently deleting the file.
That's true, I guess it's better to be safe than sorry.
>> The following patch makes todo-show display the todo file in the
>> previous window when invoked in the minibuffer. An alternative would be
>> to simply error out when invoking todo-show in the minibuffer. But I
>> think it could be useful to display the todo file while in the
>> minibuffer. Do you see a problem with this?
>
> I don't see a need for (with-selected-window)
You're right, it's unnecessary; thanks for noticing. (I had first tried
to fix it without that macro and on testing it changed the selected
window from the minibuffer to the todo-mode buffer, so that's why I
thought I needed with-selected-window, but I must have used different
functions, I can't remember anymore.)
I installed both patches in separate commits to keep the author credits
clear (with a slightly tweaked error message in your patch) and am
closing this bug. Thanks again.
Steve Berman