Re: [PATCH] git-send-email.perl: Add angle brackets to In-Reply-To if necessary

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