From: Aaron Ecay Date: Thu, 22 Dec 2011 14:24:20 +0000 (-0500) Subject: emacs: fix off-by-one bug in notmuch-show-archive X-Git-Tag: 0.11_rc1~4 X-Git-Url: https://git.cworth.org/git?a=commitdiff_plain;h=8392a7cc54c4c08c0abbf2cc2707774f504ec33f;hp=49af79cd2a3be66e482039f1ef05cf909144230e;p=notmuch emacs: fix off-by-one bug in notmuch-show-archive Text properties change between characters; prev-s-c-property-change returns the position after the change. Thus, it is still inside the invisible region. --- diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index 98db8f22..b565f336 100644 --- a/emacs/notmuch-show.el +++ b/emacs/notmuch-show.el @@ -1157,8 +1157,9 @@ current window), advance to the next open message." (ret nil)) (while (invisible-p visible-end-of-this-message) (setq visible-end-of-this-message - (previous-single-char-property-change visible-end-of-this-message - 'invisible))) + (max (point-min) + (1- (previous-single-char-property-change + visible-end-of-this-message 'invisible))))) (cond ;; Ideally we would test `end-of-this-message' against the result ;; of `window-end', but that doesn't account for the fact that