lists.zerezo.com
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/3] add '%d' pretty format specifier to show decoration
- Date: Thu, 04 Sep 2008 17:28:13 -0700
- From: Junio C Hamano <gitster@xxxxxxxxx>
- Subject: Re: [PATCH 3/3] add '%d' pretty format specifier to show decoration
Jeff King <peff@xxxxxxxx> writes:
> On Thu, Sep 04, 2008 at 11:40:03PM +0200, René Scharfe wrote:
>
>> Michael Dressel implemented an initial version and chose the letter d,
>> Junio suggested to add a leading space and parentheses.
>
> The whole series looks good to me, and I am happy if it is applied
> as-is. The only question I might raise is whether we want to use "%d"
> for this, or use something longer to anticipate a collision with other
> "d" words (I think you mentioned "describe" at one point).
How about using "%d()" for this one, so that later enhancements can
specify their features inside parentheses?
--
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