Re: Small fix for Gnus integration

[Tassilo Horn <tassilo@member.fsf.org>]

Andreas Rottmann <a.rottmann@gmx.at> writes:

Hi Andreas,

> Without the following patch, storing links to Gnus Article buffers
> fails on CVS Emacs, as they (at least in my setup -- IMAP) always
> contain a "From " line, which can't be parsed as an RFC822 header.

Could you elaborate a bit, i.e. provide such an example mail?

> diff --git a/lisp/org-gnus.el b/lisp/org-gnus.el
> index 41b1be7..4cc5adb 100644
> --- a/lisp/org-gnus.el
> +++ b/lisp/org-gnus.el
> @@ -128,6 +128,8 @@ If `org-store-link' was called with a prefix arg the meaning of
>  	   (header (with-current-buffer gnus-article-buffer
>  		     (gnus-summary-toggle-header 1)
>  		     (goto-char (point-min))
> +                     (when (looking-at "From ")
> +                       (next-line))
>  		     (mail-header-extract-no-properties)))
>  	   (from (mail-header 'from header))
>  	   (message-id (org-remove-angle-brackets

Looking at the patch, I get the impression, that it's a solution to a
problem you created on your own. ;-)

If the first line of a mail starts with "From ", I'd say it's broken.

,----[ http://cr.yp.to/immhf/field.html ]
| The first line of a field begins with a name and a colon. The name is a
| string of one or more graphical ASCII characters other than colons,
| i.e., bytes between 33 and 126 inclusive other than 58.
`----

The space has the ASCII code 32, so IMO it's not allowed as header field
name.

Bye,
Tassilo