Hello David, > I have not actually tested this, but from staring at it intensively, > it should be correct. One of the things that bit me when I tried > using this program. > > ... > - $initial_reply_to =~ s/(^\s+|\s+$)//g; > + $initial_reply_to =~ s/^\s+<?/</; > + $initial_reply_to =~ s/>?\s+$/>/; > ... some rules from rfc822: optional-field = ... / "In-Reply-To:" *(phrase / msg-id) / ... msg-id = "<" addr-spec ">" phrase = 1*word word = atom / quoted-string atom = 1*<any CHAR except specials, SPACE and CTLs> quoted-string = <"> *(qtext/quoted-pair) <"> ... With 1*... meaning "one or more" and *... meaning "zero or more". That is not all fields of In-Reply-To: must be enclosed in '<', '>'. I didn't know that before looking it up and I expect it's not very common. Even if no "phrase" is used there can be multiple msg-id's and then your patch doesn't do the right thing. But anyhow I think it's bearable to include these angle brackets in the cut-and-paste process. Best regards Uwe -- Uwe Kleine-König http://www.google.com/search?q=1+electron+mass%3D - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
