Gnus development mailing list
 help / color / mirror / Atom feed
From: Sudish Joseph <sj@eng.mindspring.net>
Subject: Re: Preformatting features of Emacs
Date: 28 Feb 1997 17:22:41 -0500	[thread overview]
Message-ID: <yviavi7co9ji.fsf@atreides.eng.mindspring.net> (raw)
In-Reply-To: Paul Franklin's message of 28 Feb 1997 12:37:13 -0800

Paul Franklin writes:
> Hmm.  I've been really happy with filladapt.  It seems to do a better
> job than either gnus-article-fill-cited-article or
> message-fill-yanked-message, but it only does one paragraph at a
> time.  The real win is that it's smart enough to notice changes in
> citation levels and treat them as paragraph boundaries even if there's
> no line separating them.

The problem is that it's paragraph-oriented.  It craps out when fed a
region containing blank lines, by converting the blank line preceding
a paragraph into a space(!).  Try putting the region across the next 3
lines and filling it.

This is paragraph a.

This is paragraph b.

This is 
paragraph c.

This gets filled as:

This is paragraph a.
 This is paragraph b.
 This is paragraph c.

This makes "W w" an ugly transform for most messages I read.

This is because it wasn't a paragraph to begin with, of course.  I
have a patch that "fixes" this, but Kyle Jones sez we shouldn't be
calling it that way in the first place.

> Will changing gnus-article-fill-cited-article and
> message-fill-yanked-message to use filladapt if it's present address
> your concerns?  If not, could you be more specific as to why this is
> insufficient?

IMO, The Right Thing to do is exactly what Per stated.  His cite stuff
is far, far, far, far (etc, you get the idea) more knowledgeable about
mail and news citation boundaries (which is what we need, not
paragraphs in the usual sense) than filladapt is.  

It can mark such regions (I expect it already does this) and stuff
like "W w" should be iterating over such regions and feeding each
region separately to the fill functions.

-Sudish


  parent reply	other threads:[~1997-02-28 22:22 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
1997-02-28 10:35 Hrvoje Niksic
1997-02-28 11:32 ` Per Abrahamsen
1997-02-28 14:12   ` Steinar Bang
1997-03-03  9:31     ` Robert Bihlmeyer
1997-02-28 16:31   ` Hrvoje Niksic
1997-02-28 15:37 ` David Blacka
1997-03-03 11:51   ` Lars Balker Rasmussen
1997-02-28 20:37 ` Paul Franklin
1997-02-28 22:17   ` Hrvoje Niksic
1997-02-28 23:26     ` Stainless Steel Rat
1997-02-28 23:58       ` Jason L Tibbitts III
1997-03-01  0:57         ` Sudish Joseph
1997-03-01  2:05           ` Alan Shutko
1997-03-01  2:53             ` Paul Franklin
1997-03-01 21:45               ` Jason L Tibbitts III
1997-03-01 23:15                 ` C. R. Oldham
1997-03-02 23:24                   ` Alan Shutko
1997-03-01  3:02         ` filladapt.el Stainless Steel Rat
1997-03-05 15:58           ` Disabling scoring for a group C. R. Oldham
1997-03-05 17:15             ` Hrvoje Niksic
1997-03-05 19:08               ` C. R. Oldham
1997-03-06 13:41                 ` Per Abrahamsen
1997-03-07  7:48                   ` Wes Hardaker
1997-03-07 12:41                     ` Per Abrahamsen
1997-03-01  5:52       ` Preformatting features of Emacs Hrvoje Niksic
1997-03-01  0:29     ` Sudish Joseph
1997-02-28 22:22   ` Sudish Joseph [this message]
1997-02-28 22:47     ` Per Abrahamsen
1997-02-28 23:56       ` Sudish Joseph
1997-03-01  0:21         ` Stephen Peters
1997-02-28 23:59       ` Sudish Joseph
1997-03-01  1:04         ` Lars Magne Ingebrigtsen
1997-03-01  1:53           ` Sudish Joseph
1997-03-01  2:13           ` Sudish Joseph
1997-03-01  2:17           ` Steven L Baur
1997-03-01  3:06             ` Lars Magne Ingebrigtsen

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=yviavi7co9ji.fsf@atreides.eng.mindspring.net \
    --to=sj@eng.mindspring.net \
    /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).