Gnus development mailing list
 help / color / mirror / Atom feed
From: "Stefan Monnier" <monnier+gnu/emacs/pretest@RUM.cs.yale.edu>
Cc: "Stefan Monnier" <monnier+gnu/emacs/pretest@RUM.cs.yale.edu>,
	ding@gnus.org,
	GNU Emacs pretest bug list <emacs-pretest-bug@gnu.org>
Subject: Re: article body filling: fill-prefix too long for specified width
Date: Mon, 25 Mar 2002 16:52:55 -0500	[thread overview]
Message-ID: <200203252152.g2PLqtm14363@rum.cs.yale.edu> (raw)
In-Reply-To: <m3n0wwz8lr.fsf@Janik.cz>

>    From: "Stefan Monnier" <monnier+gnu/emacs/pretest@rum.cs.yale.edu>
>    Date: Mon, 25 Mar 2002 11:12:28 -0500
> 
> Hi Stefan,
> 
>    > > I have received attached e-mail, which can not be displayed in Gnus (from
>    > > CVS) running in current CVS Emacs. The message
>    > > 
>    > >      fill-prefix too long for specified width
>    > > 
>    > > is shown. fill-column is 75 in my setup. Additional details on
>    > > request. I still have this article so I can test your changes.
>    > 
>    > It must be a bug in my changes in fill.el.
>    > I can't look at it just now, so I'll get back to you later,
> 
> I do not know where the bug is, but this change from Katsumi Yamaoka
> <yamaoka@jpl.org> fixed this particular problem:
> 
> --- cut here ---
> I'm not sure of myself but I think the value of `fill-prefix'
> should be controlled by flow-fill.el, however it will be altered
> by `fill-region-as-paragraph' if it isn't specified.  Here's a
> patch for a tentative plan:
> 
> --- flow-fill.el~	2002-01-14 22:01:23.000000000 +0000
> +++ flow-fill.el	2002-03-25 12:55:36.039833946 +0000
> @@ -134,7 +134,8 @@
>  	    (end-of-line))
>  	  (unless sig
>  	    (let ((fill-prefix (when quote (concat quote " ")))
> -		  (fill-column (eval fill-flowed-display-column)))
> +		  (fill-column (eval fill-flowed-display-column))
> +		  adaptive-fill-mode)
>  	      (fill-region (fill-flowed-point-at-bol)
>  			   (min (1+ (fill-flowed-point-at-eol)) (point-max))
>  			   'left 'nosqueeze))))))))

This indeed will prevent the buggy code from running.
But it doesn't fix the bug.  Of course, the patch might still be correct
for flow-fill.el, I don't know.


	Stefan




      reply	other threads:[~2002-03-25 21:52 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-03-24 17:33 Pavel Janík
2002-03-25 12:57 ` Katsumi Yamaoka
2002-03-25 16:12 ` Stefan Monnier
2002-03-25 20:38   ` Pavel Janík
2002-03-25 21:52     ` Stefan Monnier [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200203252152.g2PLqtm14363@rum.cs.yale.edu \
    --to=monnier+gnu/emacs/pretest@rum.cs.yale.edu \
    --cc=ding@gnus.org \
    --cc=emacs-pretest-bug@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).