[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#20663: page.el (forward-page): Avoid skipping pages
From: |
Pierre Neidhardt |
Subject: |
bug#20663: page.el (forward-page): Avoid skipping pages |
Date: |
Sun, 10 Apr 2016 08:29:24 +0700 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
I did not expect this patch to be so confusing, but let me clarify the above
questions:
- `forward-page' is called by `narrow-to-page', which is bound to 'C-x n p' by
default.
- All the patch does it make the code consistent with its comments, that is:
;; In case the page-delimiter matches the null string,
;; don't find a match without moving.
- If you try the recipe (I just did on Emacs 24.5.1, don't have time to check
now on upstream), you'll see that a page gets skipped, which is not the desired
behaviour.
- As for side effects, there might be some, althought I haven't noticed anything
in a year of use. If there is, then it is a bug in the caller.
--
Pierre Neidhardt
- bug#20663: page.el (forward-page): Avoid skipping pages, Marcin Borkowski, 2016/04/09
- bug#20663: page.el (forward-page): Avoid skipping pages, Eli Zaretskii, 2016/04/09
- bug#20663: page.el (forward-page): Avoid skipping pages, Marcin Borkowski, 2016/04/09
- bug#20663: page.el (forward-page): Avoid skipping pages, Eli Zaretskii, 2016/04/09
- bug#20663: page.el (forward-page): Avoid skipping pages,
Pierre Neidhardt <=
- bug#20663: page.el (forward-page): Avoid skipping pages, Marcin Borkowski, 2016/04/11
- bug#20663: page.el (forward-page): Avoid skipping pages, Eli Zaretskii, 2016/04/11
- bug#20663: page.el (forward-page): Avoid skipping pages, Marcin Borkowski, 2016/04/13
- bug#20663: page.el (forward-page): Avoid skipping pages, John Mastro, 2016/04/13
- bug#20663: page.el (forward-page): Avoid skipping pages, Marcin Borkowski, 2016/04/13
- bug#20663: page.el (forward-page): Avoid skipping pages, Marcin Borkowski, 2016/04/16
- bug#20663: page.el (forward-page): Avoid skipping pages, Eli Zaretskii, 2016/04/16
- bug#20663: page.el (forward-page): Avoid skipping pages, Marcin Borkowski, 2016/04/20
- bug#20663: page.el (forward-page): Avoid skipping pages, Pierre Neidhardt, 2016/04/27