GNU bug report logs - #47338
28.0.50; Unexpected error in fill-region

Previous Next

Full log

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

From: Evgeny Zajcev <lg.zevlg <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 47338 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#47338: 28.0.50; Unexpected error in fill-region
Date: Thu, 25 Mar 2021 21:27:43 +0300

[Message part 1 (text/plain, inline)]

ср, 24 мар. 2021 г. в 12:24, Lars Ingebrigtsen <larsi <at> gnus.org>:

> Eli Zaretskii <eliz <at> gnu.org> writes:
>
> > This is the result of fixing bug#45720.  In the above scenario, the
> > first time through the loop LINEBEG in the fragment below is set to
> > EOB+1 instead of the point where the fill-prefix ends:
> >
> >       (let ((first t)
> >               linebeg)
> >         (while (< (point) to)
> >             ;; On the first line, there may be text in the fill prefix
> >             ;; zone.  In that case, don't consider that area when
> >             ;; trying to find a place to put a line break (bug#45720).
> >             (if (not first)
> >               (setq linebeg (point))
> >               (setq first nil
> >                     linebeg (+ (point) (length fill-prefix)))) <<<<<<<<<<
> >
> > CC'ing Lars who made that change.
>
> Right...  the change tried to fix the case where there's a pre-computed
> fill prefix being passed in.  In this bug, however, the prefix is
> computed, and this logic shouldn't be done here.  I've now fixed this in
> Emacs 28, but I'm still not quite sure about the original fix...
>
>
Thanks! this works perfectly for me

-- 
lg

[Message part 2 (text/html, inline)]

Previous Next

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