From: Jonathan Nieder Date: Mon, 28 Jan 2013 03:39:21 GMT Subject: Re: [PATCH v3 11/11] Unify appending signoff in format-patch, commit and sequencer Message-ID: <20130128033921.GP8206@elie.Belkin> In-Reply-To: <1359335515-13818-12-git-send-email-drafnel@gmail.com> Brandon Casey wrote: > --- a/log-tree.c > +++ b/log-tree.c [...] > @@ -208,94 +207,6 @@ void show_decorations(struct rev_info *opt, struct commit *commit) > putchar(')'); > } > > -/* > - * Search for "^[-A-Za-z]+: [^@]+@" pattern. It usually matches > - * Signed-off-by: and Acked-by: lines. > - */ That's stricter than the test from sequencer.c. Maybe it's worth stealing to avoid false positives?