[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulat
From: |
Mark Oteiza |
Subject: |
Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulation |
Date: |
Mon, 05 Dec 2016 21:03:04 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (gnu/linux) |
Fixing C-c C-a to DTRT is great, thanks, but I don't think the
off-by-one navigation changes to "n" and "p" (diff-hunk-next,
diff-hunk-prev) make sense. While it may have made fixing the issues
mentioned in the commit message easier, the changes to what "n" and "p"
do at the beginning and end of a diff are not documented, and I didn't
see any discussion of it in the associated bug.
I contend that the new behavior is inconsistent with the behavior of
other outline/thing-with-headers type things in Emacs. outline-mode,
org-mode, and rst-mode are the first ones that come to mind.
It's also not clear how the introduced oddity with auto-refine is going
to be resolved, unless a way is found to autorefine the first hunk
without there being any user interaction. Then opening a diff has
inconsistent auto-refining from the start.
- Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulation,
Mark Oteiza <=
- Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulation, Dima Kogan, 2016/12/09
- Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulation, Dmitry Gutov, 2016/12/10
- Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulation, Dima Kogan, 2016/12/10
- Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulation, Dmitry Gutov, 2016/12/11
- Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulation, Dima Kogan, 2016/12/12
- Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulation, Dmitry Gutov, 2016/12/15
- Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulation, Tino Calancha, 2016/12/19
- Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulation, Dima Kogan, 2016/12/20
- Re: [Emacs-diffs] master 2c8a7e5: Improve diff-mode navigation/manipulation, Dima Kogan, 2016/12/25