Re: [PATCH next v2] git-notes: add test case for multi-line notes

From: Jeff King <peff@peff.net>
Date: Jan 14, 2009, 17:13 UTC
Message-ID: <20090114171315.GA16063@coredump.intra.peff.net>
In-Reply-To: <200901141109.56580.bss@iguanasuicide.net>

On Wed, Jan 14, 2009 at 11:09:52AM -0600, Boyd Stephen Smith Jr. wrote:

Show 8 quoted lines

> >  printf %s "$MSG" ?
> 
> On my box
> $ printf '%s\n' '\n'
> \n
> $
> 
> He wants '\n' in $MSG to be expanded, and what you gave doesn't do that.

Oh, sorry. That's what I get for not reading his patch carefully.

It looks like all of the input is statically included in the test script. While I think it is nice to be defensive, it is probably simplest to just assume there is no '%' in this case (which we can verify by reading the script).

-Peff