undesirable feature report -- invalid wrapped headers

undesirable feature report -- invalid wrapped headers

[lconrad]

When I reply to a message with a long From field, such as:

From: Timothy Holland - Supply Program Manager <tholland@tunnel.East.Sun.COM>

I find that the To field that gnus produces is sometimes like this:

To: Timothy Holland - Supply Program Manager 
<tholland@tunnel.East.Sun.COM>

Unless I edit this, gnus has trouble sending the mail, because the To
header (which isn't continued onto the second line, because it doesn't
include a space) doesn't include the mail address.

Can wrapped headers be wrapped with a fill prefix of at least one
space?  

Or is there some more complicated alternative involving a hook that's
executed after the headers are wrapped and before R and F yank in the
replied to text?  And if so, can the way to use this be documented?

-- 
Laura (mailto:lconrad@world.std.com , http://www.world.std.com/~lconrad/ )
(617) 661-8097	fax: (801) 365-6574 
233 Broadway, Cambridge, MA 02139

Re: undesirable feature report -- invalid wrapped headers

[Lars Magne Ingebrigtsen]

lconrad@world.std.com writes:

> From: Timothy Holland - Supply Program Manager <tholland@tunnel.East.Sun.COM>
> 
> I find that the To field that gnus produces is sometimes like this:
> 
> To: Timothy Holland - Supply Program Manager 
> <tholland@tunnel.East.Sun.COM>

That's not good.  When does Gnus do this?  I've tried (very, very
briefly) to reproduce this, but things seem to work for me...

-- 
(domestic pets only, the antidote for overdose, milk.)
   larsi@gnus.org * Lars Magne Ingebrigtsen

Re: undesirable feature report -- invalid wrapped headers

[Laura Conrad]

>>>>> "Lars" == Lars Magne Ingebrigtsen <larsi@gnus.org> writes:

    Lars> lconrad@world.std.com writes:
    >> From: Timothy Holland - Supply Program Manager <tholland@tunnel.East.Sun.COM>
    >> 
    >> I find that the To field that gnus produces is sometimes like this:
    >> 
    >> To: Timothy Holland - Supply Program Manager 
    >> <tholland@tunnel.East.Sun.COM>

    Lars> That's not good.  When does Gnus do this?  I've tried (very, very
    Lars> briefly) to reproduce this, but things seem to work for me...

When you're using auto-fill, and it insists on filling the headers.
Here's everything from my .emacs that looks relevant:

;; Auto fill in mail
(turn-on-auto-fill)
(add-hook 'text-mode-hook
            '(lambda () (auto-fill-mode 1)))

;; Auto fill in mail
(setq mail-mode-hook 'turn-on-auto-fill)

;;filladapt goes with supercite
 (require 'filladapt)
    (setq-default filladapt-mode t)

I think we need something that causes gnus to either not fill headers at 
all or to use an appropriate fill prefix.

-- 
Laura (mailto:lconrad@world.std.com , http://www.world.std.com/~lconrad/ )
(617) 661-8097	fax: (801) 365-6574 
233 Broadway, Cambridge, MA 02139

Re: undesirable feature report -- invalid wrapped headers

[Kai Großjohann]

Laura Conrad <lconrad@world.std.com> writes:

> When you're using auto-fill, and it insists on filling the headers.

Maybe message-mode should set auto-fill-inhibit-regexp to a useful
value by default?

kai
-- 
Beware of flying birch trees.

Re: undesirable feature report -- invalid wrapped headers

[Lars Magne Ingebrigtsen]

Laura Conrad <lconrad@world.std.com> writes:

> When you're using auto-fill, and it insists on filling the headers.

Right.

> I think we need something that causes gnus to either not fill headers at 
> all or to use an appropriate fill prefix.

I don't know much about the filling functions -- how does one do
context-sensitive filling?  That is, use one set of filling in the
headers and one in the body?

-- 
(domestic pets only, the antidote for overdose, milk.)
   larsi@gnus.org * Lars Magne Ingebrigtsen

Re: undesirable feature report -- invalid wrapped headers

[Lars Magne Ingebrigtsen]

Kai.Grossjohann@CS.Uni-Dortmund.DE (Kai Großjohann) writes:

> Maybe message-mode should set auto-fill-inhibit-regexp to a useful
> value by default?

Please submit a patch with a useful value for this variable.  :-)

-- 
(domestic pets only, the antidote for overdose, milk.)
   larsi@gnus.org * Lars Magne Ingebrigtsen

Re: undesirable feature report -- invalid wrapped headers

[Kai Großjohann]

Lars Magne Ingebrigtsen <larsi@gnus.org> writes:

> Kai.Grossjohann@CS.Uni-Dortmund.DE (Kai Großjohann) writes:
> 
> > Maybe message-mode should set auto-fill-inhibit-regexp to a useful
> > value by default?
> 
> Please submit a patch with a useful value for this variable.  :-)

No problem -- except that I'm not sure that the regexp used in
font-locking, "^\\([A-Z][^: \n\t]+:\\)" (I think), is good enough for
this.  It produces false matches.

But what the heck.

kai
-- 
Beware of flying birch trees.

Re: undesirable feature report -- invalid wrapped headers

[Kai Großjohann]

Lars Magne Ingebrigtsen <larsi@gnus.org> writes:

> Please submit a patch with a useful value for this variable.  :-)

Whee.  This one was easy.  I even tested it twice :-)

But: there is a potential of false matches, like this line for example.  It is not auto-filled.  As you can see.

kai
-- 
Beware of flying birch trees.

diff -u -r5.88 message.el
--- lisp/message.el	2000/04/20 19:43:02	5.88
+++ lisp/message.el	2000/04/20 21:52:37
@@ -1462,6 +1477,8 @@
   (setq adaptive-fill-first-line-regexp
 	(concat "[ \t]*[-a-z0-9A-Z]*\\(>[ \t]*\\)+[ \t]*\\|"
 		adaptive-fill-first-line-regexp))
+  (make-local-variable 'auto-fill-inhibit-regexp)
+  (setq auto-fill-inhibit-regexp "^[A-Z][^: \n\t]+:")
   (mm-enable-multibyte)
   (make-local-variable 'indent-tabs-mode) ;Turn off tabs for indentation.
   (setq indent-tabs-mode nil)

Re: undesirable feature report -- invalid wrapped headers

[Laura Conrad]

>>>>> "Kai" == Kai Großjohann <Kai.Grossjohann@CS.Uni-Dortmund.DE> writes:

    Kai> But: there is a potential of false matches, like this line
    Kai> for example.  It is not auto-filled.  As you can see. 

But if I go to the end of it and hit a space, then it fills
correctly.  As you can see.

-- 
Laura (mailto:lconrad@world.std.com , http://www.world.std.com/~lconrad/ )
(617) 661-8097	fax: (801) 365-6574 
233 Broadway, Cambridge, MA 02139

Re: undesirable feature report -- invalid wrapped headers

[Lars Magne Ingebrigtsen]

Kai.Grossjohann@CS.Uni-Dortmund.DE (Kai Großjohann) writes:

> Whee.  This one was easy.  I even tested it twice :-)

Thanks for the patch; I've applied it to Gnus v5.8.5.

-- 
(domestic pets only, the antidote for overdose, milk.)
   larsi@gnus.org * Lars Magne Ingebrigtsen