Re: Fwd: [PATCH] MAINTAINERS: Quote non-alphanumeric email addresses

[Harry Wei <jiaweiwei.xiyou@gmail.com>]

On Mon, Feb 21, 2011 at 12:59:16PM +0800, jiaweiwei wrote:
> ---------- Forwarded message ----------
> From: Joe Perches <joe@perches.com>
> Date: 2011/2/21
> Subject: Re: [PATCH] MAINTAINERS: Quote non-alphanumeric email addresses
> To: Harry Wei <jiaweiwei.xiyou@gmail.com>
> Cc: davem@davemloft.net, akpm@linux-foundation.org,
> linux-kernel@vger.kernel.org, greg@kroah.com, rdunlap@xenotime.net,
> pavel@ucw.cz
> 
> 
> On Sun, 2011-02-20 at 21:59 +0800, Harry Wei wrote:
> > On Sat, Feb 19, 2011 at 08:46:54PM -0800, Joe Perches wrote:
> > > Add quotes around email address with periods and commas.
> > Hmm, what about some other conditions except
> > for periods and commas in maintainers' M:.
> > Like following:
> > 1> Character '-' in M:
> 
> Dashes are not an issue.
Get it.
> 
> > 2> No name of the M:
> > M:      support@lsi.com
> 
> Names and angle brackets are optional.
Ok.
> 
> > 3> Messy code in M:
> > M:      Benoît Cousson <b-cousson@ti.com>
> 
> 8 bit characters _might_ be an issue, but
> no one has reported any problems yet.
Get it. 
> 
> I think you should read and understand rfc 2822
> before continuing with this.
Thanks, i will read it :)
> 
> Utilities like git send-email and get_maintainer.pl
> generally provide email address parsing for only a
> subset of all valid address styles.
> 
> As these utilities are done mostly by regex, not
> all valid email addresses are supported because the
> regex validations get ridiculously complicated.
> 
> There are some "monster" regex classes that attempt
> to support a greater range of valid email address
> styles, but the pain of implementation generally
> isn't worth it.
> 
> You're free to try to improve it of course, but I
> think that modifications to the address styles
> currently used in MAINTAINERS to remove "special"
> address style handling isn't necessary.
Get it, thanks.

Best Regards.
Harry Wei.
> 
> 
> 
> 
> -- 
>  do kernel