Re: [PATCH] pretty.c: add %z specifier.

From: Jeff King <peff@peff.net>
Date: Mar 21, 2008, 04:51 UTC
Message-ID: <20080321045137.GA5563@coredump.intra.peff.net>
In-Reply-To: <7veja4u1gv.fsf@gitster.siamese.dyndns.org>

On Thu, Mar 20, 2008 at 09:48:16PM -0700, Junio C Hamano wrote:

Show 9 quoted lines

> > +	case 'z':		/* null */
> > +		strbuf_addch(sb, '\0');
> > +		return 1;
> >  	}
> >
> >  	/* these depend on the commit */
> 
> I do not like this at all.  Why aren't we doing %XX (2 hexadecimal digits
> for an octet)?

Because %ad is already taken? :)

%x* is still available, though, so maybe %x00?

-Peff