GNU bug report logs - #13067
Using `gnus-treat-fill-long-lines': many problems

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 13067 in the body.
You can then email your comments to 13067 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: Michael Heerdegen <michael_heerdegen <at> web.de>
To: submit <at> debbugs.gnu.org (The Gnus Bugfixing Girls + Boys)
Subject: Using `gnus-treat-fill-long-lines': many problems
Date: Mon, 03 Dec 2012 14:50:07 +0100

Hi,

I wanted to set `gnus-treat-fill-long-lines' on a per-group basis, after
I had read "4.4 Customizing Articles" in the manual.  It took 3 hours to
have a look and find out several reasons why this doesn't work as
intended.  These reasons are:

1.  It doesn't work to use a binding to `gnus-treat-fill-long-lines' as
a group paramenter.  It is just ignored.  It is even ignored if I push
gnus-treat-fill-long-lines to `gnus-newsgroup-variables'.  The value is
always nil, in any buffer.

Something like that should be possible.  It is very likely that users
will want to configure such a treatment setting per group.


2.  Only a (global) value of t for `gnus-treat-fill-long-lines' has an
effect.  Setting it to an integer, which is also allowed according to
the documentation, doesn't work for me.


3.  If I want the setting to take effect for html messages, which is a
IMHO quite common wish, if have to do this:

   (push "text/html" gnus-article-treat-types)

Note that `gnus-article-treat-types' is not a user option, and its
documentation consist of three words.


Maybe I overlooked something or tried the wrong things - then please
tell me.  But I think this could work better.


Thanks,

Michael.



Gnus v5.13
GNU Emacs 24.2.90.1 (x86_64-unknown-linux-gnu, GTK+ Version 2.24.10)
 of 2012-11-30 on drachen
200 news.gmane.org InterNetNews NNRP server INN 2.5.1 ready (posting ok)
101 Capability list:
VERSION 2
IMPLEMENTATION INN 2.5.1
AUTHINFO USER
HDR
LIST ACTIVE ACTIVE.TIMES DISTRIB.PATS HEADERS NEWSGROUPS OVERVIEW.FMT
OVER
POST
READER
STARTTLS
.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Michael Heerdegen <michael_heerdegen <at> web.de>
Cc: 13067 <at> debbugs.gnu.org
Subject: Re: bug#13067: Using `gnus-treat-fill-long-lines': many problems
Date: Wed, 26 Dec 2012 16:37:19 +0100

Michael Heerdegen <michael_heerdegen <at> web.de> writes:

> 1.  It doesn't work to use a binding to `gnus-treat-fill-long-lines' as
> a group paramenter.  It is just ignored.

I tried setting it in a group parameter, and the value seems to be
respected.  How are you setting it?

> 2.  Only a (global) value of t for `gnus-treat-fill-long-lines' has an
> effect.  Setting it to an integer, which is also allowed according to
> the documentation, doesn't work for me.

Works for me as demonstrated by `M-x debug-on-entry RET
gnus-article-fill-cited-long-lines RET'.

> 3.  If I want the setting to take effect for html messages, which is a
> IMHO quite common wish, if have to do this:
>
>    (push "text/html" gnus-article-treat-types)

Yes, that should probably be on the list.  I've now added it.

-- 
(domestic pets only, the antidote for overdose, milk.)
  http://lars.ingebrigtsen.no  *  Lars Magne Ingebrigtsen

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

From: Michael Heerdegen <michael_heerdegen <at> web.de>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 13067 <at> debbugs.gnu.org
Subject: Re: bug#13067: Using `gnus-treat-fill-long-lines': many problems
Date: Wed, 26 Dec 2012 17:59:26 +0100

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> Michael Heerdegen <michael_heerdegen <at> web.de> writes:
>
> > 1.  It doesn't work to use a binding to `gnus-treat-fill-long-lines' as
> > a group paramenter.  It is just ignored.
>
> I tried setting it in a group parameter, and the value seems to be
> respected.  How are you setting it?

I think it only didn't work because of my bad setup (shr-width ==
1000).  Please ignore.

> > 2.  Only a (global) value of t for `gnus-treat-fill-long-lines' has an
> > effect.  Setting it to an integer, which is also allowed according to
> > the documentation, doesn't work for me.
>
> Works for me as demonstrated by `M-x debug-on-entry RET
> gnus-article-fill-cited-long-lines RET'.

Good.  I guess this was also due to my bogus setup.  Please ignore.

> > 3.  If I want the setting to take effect for html messages, which is a
> > IMHO quite common wish, if have to do this:
> >
> >    (push "text/html" gnus-article-treat-types)
>
> Yes, that should probably be on the list.  I've now added it.

Ok - but not sure anymore if this correct.  Normally, the filling is
done by the html renderer, I guess.  Is it now possible that filling is
done twice, maybe with two different line lengths?


Regards,

Michael.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Michael Heerdegen <michael_heerdegen <at> web.de>
Cc: 13067 <at> debbugs.gnu.org
Subject: Re: bug#13067: Using `gnus-treat-fill-long-lines': many problems
Date: Wed, 26 Dec 2012 18:04:15 +0100

Michael Heerdegen <michael_heerdegen <at> web.de> writes:

> Ok - but not sure anymore if this correct.  Normally, the filling is
> done by the html renderer, I guess.  Is it now possible that filling is
> done twice, maybe with two different line lengths?

Yes.  But the same is true for all other article treatment functions --
they should probably be run on HTML parts, too.

We'll see how this works in practice.

-- 
(domestic pets only, the antidote for overdose, milk.)
  http://lars.ingebrigtsen.no  *  Lars Magne Ingebrigtsen

Previous Next

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