GNU bug report logs - #20663
page.el (forward-page): Avoid skipping pages

Previous Next

Full log

Message #26 received at 20663 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Marcin Borkowski <mbork <at> mbork.pl>
Cc: 20663 <at> debbugs.gnu.org, ambrevar <at> gmail.com
Subject: Re: bug#20663: page.el (forward-page): Avoid skipping pages
Date: Mon, 11 Apr 2016 18:35:03 +0300

> From: Marcin Borkowski <mbork <at> mbork.pl>
> Cc: Eli Zaretskii <eliz <at> gnu.org>,  20663 <at> debbugs.gnu.org
> Date: Mon, 11 Apr 2016 12:20:07 +0200
> 
> I spent a few minutes on this with Edebug and Git today.

Thanks.

> > - 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.
> 
> Seems ok.
> 
> > - 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.
> 
> Condirmed.  And it doesn't happen with the patch installed.

Can one of you please explain why the original code misbehaves?

> For instance, when the point is -!-, is the point on the first or
> second page here?  Emacs with and without the patch has different
> opinions on that.

Good point.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.