[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Orgmode] Re: Small fix for Gnus integration
From: |
Tassilo Horn |
Subject: |
[Orgmode] Re: Small fix for Gnus integration |
Date: |
Mon, 06 Jul 2009 20:16:32 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1.50 (gnu/linux) |
Andreas Rottmann <address@hidden> 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