From: Felipe Contreras Date: Sat, 04 Feb 2012 21:04:23 GMT Subject: Re: [PATCH 2/3] t: mailmap: add 'git blame -e' tests Message-ID: In-Reply-To: <20120204201027.GE22928@burratino> On Sat, Feb 4, 2012 at 10:10 PM, Jonathan Nieder wrote: > Since I didn't receive a copy of the cover letter or patch 1, I don't > know what this is intended to test _for_.  Good --- I can more easily > convey the reaction of future readers who do not necessarily know the > context in which the patch was written (and the commit message does > not seem to say). > > Looking above, I see > >  - a lone comment "git blame".  What is it trying to tell me?  I guess >   you copy/pasted it, but is there any purpose to it? > >  - a test asserting the claim "Blame output (complex mapping)".  This >   title is identical to the test before.  I have no idea what this is >   about. Look at the title: add 'git blame -e' tests s/blame/blame -e/ -- Felipe Contreras