On Sat, Feb 4, 2012 at 10:10 PM, Jonathan Nieder <jrnieder@xxxxxxxxx> 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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html