Re: gnus-split-references tweak

[prj@po.cwru.edu (Paul Jarc)]

Jesper Harder <harder@ifa.au.dk> wrote:
> prj@po.cwru.edu (Paul Jarc) writes:
>> -      (when (string-match "<[^> \t]+>\\'" references)
>> -       (match-string 0 references)))))
>> +      (when (string-match "\(<[^<]+[^< \t]\)[ \t]*\\'" references)
>> +       (match-string 1 references)))))
>
> You missed a couple of backslashes in the regexp, it should be
>
>   "\\(<[^<]+[^< \t]\\)[ \t]*\\'"

Not my day.  And yet somehow it seems to work anyway.  Weird.

> I think this one:
>
>       (when (string-match "\\(<[^>]+>\\)[ \t]*\\'" references)
> 	(match-string 1 references)))))
>
> is more efficient -- the performance penalty was only about 2%.

Yeah, that should be ok.  I've sometimes seen References fields
truncated in the missle of a Message-ID, and this regexp won't find
the last, partial one, but that's ok because the partial one wouldn't
have matched any message's Message-ID anyway.


paul