threads / discuss / 30549

Are author names ending with a dot invalid?

Subject: Are author names ending with a dot invalid?

## tl;dr

2 messages between May 18, 2012 and May 19, 2012.

replies: 1people: 2as markdown or json

Felipe Contreras· May 18, 2012, 00:14 UTC · lore
Hi,

It looks like authors ending with a dot ('Foo Jr. <foo@bar.com>') are converted without a dot ('Foo Jr <foo@bar.com>') is that intentional?

Cheers.
-- 
Felipe Contreras
Jeff King· May 19, 2012, 00:23 UTC · re: Felipe Contreras · lore

Re: Are author names ending with a dot invalid?

On Fri, May 18, 2012 at 02:14:57AM +0200, Felipe Contreras wrote:
> It looks like authors ending with a dot ('Foo Jr. <foo@bar.com>') are
> converted without a dot ('Foo Jr <foo@bar.com>') is that intentional?

I was just digging in this code today, and I noticed the same thing. The culprit is ident.c:copy, which strips magic characters from the beginning, end, and middle. Except that the set of magic characters is different at the edges (see ident.c:crud), and in the middle ("<", ">", "\n").

This dates all the way back to 6aa33f4 (Abstract out the "name <email> date" handling of commit-tree.c, 2005-07-12), which in turn was adapting code from earlier (though it also has the issue).

So I think it's intentional, but I also think it's wrong, and it's simply that nobody has complained about it, because it's not that common.

-Peff

← back to recent threads