* Another todo-mode bug fix, ok for emacs-25? @ 2016-05-04 13:59 Stephen Berman 2016-05-04 15:18 ` Eli Zaretskii 0 siblings, 1 reply; 5+ messages in thread From: Stephen Berman @ 2016-05-04 13:59 UTC (permalink / raw) To: emacs-devel In bug#23447 I show how to reproduce a todo-mode bug I encountered involving archived items and provide a fix for it. The bug does not result in data loss or file corruption, but changes the behavior of todo-show in a way that can only be repaired by resetting internal variables or unloading and reloading todo-mode.el. The patch has no effect outside of todo-mode, and I've tested all uses of the function involved within todo-mode and encountered no problems, so is it all right for me to commit the fix to emacs-25? Steve Berman ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Another todo-mode bug fix, ok for emacs-25? 2016-05-04 13:59 Another todo-mode bug fix, ok for emacs-25? Stephen Berman @ 2016-05-04 15:18 ` Eli Zaretskii 2016-05-04 17:46 ` Stephen Berman 0 siblings, 1 reply; 5+ messages in thread From: Eli Zaretskii @ 2016-05-04 15:18 UTC (permalink / raw) To: Stephen Berman; +Cc: emacs-devel > From: Stephen Berman <stephen.berman@gmx.net> > Date: Wed, 04 May 2016 15:59:11 +0200 > > In bug#23447 I show how to reproduce a todo-mode bug I encountered > involving archived items and provide a fix for it. The bug does not > result in data loss or file corruption, but changes the behavior of > todo-show in a way that can only be repaired by resetting internal > variables or unloading and reloading todo-mode.el. The patch has no > effect outside of todo-mode, and I've tested all uses of the function > involved within todo-mode and encountered no problems, so is it all > right for me to commit the fix to emacs-25? We don't want to release Emacs 25.1 with a buggy todo-mode, so the fact the change is local helps, but not enough. Please try to convince us that the change itself is safe. For example, is the affected code called in different use cases than the one you anted to fix, and if so, did you make sure those other use cases still work OK? Thanks. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Another todo-mode bug fix, ok for emacs-25? 2016-05-04 15:18 ` Eli Zaretskii @ 2016-05-04 17:46 ` Stephen Berman 2016-05-04 19:06 ` Eli Zaretskii 0 siblings, 1 reply; 5+ messages in thread From: Stephen Berman @ 2016-05-04 17:46 UTC (permalink / raw) To: Eli Zaretskii; +Cc: emacs-devel On Wed, 04 May 2016 18:18:39 +0300 Eli Zaretskii <eliz@gnu.org> wrote: >> From: Stephen Berman <stephen.berman@gmx.net> >> Date: Wed, 04 May 2016 15:59:11 +0200 >> >> In bug#23447 I show how to reproduce a todo-mode bug I encountered >> involving archived items and provide a fix for it. The bug does not >> result in data loss or file corruption, but changes the behavior of >> todo-show in a way that can only be repaired by resetting internal >> variables or unloading and reloading todo-mode.el. The patch has no >> effect outside of todo-mode, and I've tested all uses of the function >> involved within todo-mode and encountered no problems, so is it all >> right for me to commit the fix to emacs-25? > > We don't want to release Emacs 25.1 with a buggy todo-mode, so the > fact the change is local helps, but not enough. Please try to > convince us that the change itself is safe. For example, is the > affected code called in different use cases than the one you anted to > fix, and if so, did you make sure those other use cases still work OK? The code changes are confined to the function todo-jump-to-category, which can be invoked interactively and is also called noninteractively from two other functions, todo-jump-to-archive-category and todo-insert-category-line. The last of these uses is what give rise to the bug, under the conditions given in the bug report (archived items, non-nil todo-skip-archived-categories). With the fix applied, I've also tested this use when these conditions are not met, and I've tested the other noninteractive use and the interactive uses (the command can be called not only within Todo mode but also from a non-Todo mode buffer). In all of my tests with the patched code, the behavior was as expected. Unfortunately, I don't have a test suite for todo-mode (I hope to develop one, but that's a long-term goal). Steve Berman ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Another todo-mode bug fix, ok for emacs-25? 2016-05-04 17:46 ` Stephen Berman @ 2016-05-04 19:06 ` Eli Zaretskii 2016-05-04 19:55 ` Stephen Berman 0 siblings, 1 reply; 5+ messages in thread From: Eli Zaretskii @ 2016-05-04 19:06 UTC (permalink / raw) To: Stephen Berman; +Cc: emacs-devel > From: Stephen Berman <stephen.berman@gmx.net> > Cc: emacs-devel@gnu.org > Date: Wed, 04 May 2016 19:46:27 +0200 > > The code changes are confined to the function todo-jump-to-category, > which can be invoked interactively and is also called noninteractively > from two other functions, todo-jump-to-archive-category and > todo-insert-category-line. The last of these uses is what give rise to > the bug, under the conditions given in the bug report (archived items, > non-nil todo-skip-archived-categories). With the fix applied, I've also > tested this use when these conditions are not met, and I've tested the > other noninteractive use and the interactive uses (the command can be > called not only within Todo mode but also from a non-Todo mode buffer). > In all of my tests with the patched code, the behavior was as expected. Thanks, I'm sold. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Another todo-mode bug fix, ok for emacs-25? 2016-05-04 19:06 ` Eli Zaretskii @ 2016-05-04 19:55 ` Stephen Berman 0 siblings, 0 replies; 5+ messages in thread From: Stephen Berman @ 2016-05-04 19:55 UTC (permalink / raw) To: Eli Zaretskii; +Cc: emacs-devel On Wed, 04 May 2016 22:06:10 +0300 Eli Zaretskii <eliz@gnu.org> wrote: >> From: Stephen Berman <stephen.berman@gmx.net> >> Cc: emacs-devel@gnu.org >> Date: Wed, 04 May 2016 19:46:27 +0200 >> >> The code changes are confined to the function todo-jump-to-category, >> which can be invoked interactively and is also called noninteractively >> from two other functions, todo-jump-to-archive-category and >> todo-insert-category-line. The last of these uses is what give rise to >> the bug, under the conditions given in the bug report (archived items, >> non-nil todo-skip-archived-categories). With the fix applied, I've also >> tested this use when these conditions are not met, and I've tested the >> other noninteractive use and the interactive uses (the command can be >> called not only within Todo mode but also from a non-Todo mode buffer). >> In all of my tests with the patched code, the behavior was as expected. > > Thanks, I'm sold. Thanks, committed as emacs-25 0932b94. Steve Berman ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-05-04 19:55 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-05-04 13:59 Another todo-mode bug fix, ok for emacs-25? Stephen Berman 2016-05-04 15:18 ` Eli Zaretskii 2016-05-04 17:46 ` Stephen Berman 2016-05-04 19:06 ` Eli Zaretskii 2016-05-04 19:55 ` Stephen Berman
Code repositories for project(s) associated with this external index https://git.savannah.gnu.org/cgit/emacs.git https://git.savannah.gnu.org/cgit/emacs/org-mode.git This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.