Re: [PATCH 2/2] pretty: add %D format specifier

2014-09-11 Thread Junio C Hamano
Harry Jeffery writes: > On 11/09/14 17:56, Junio C Hamano wrote: >> Because patch 1/2 alone does not make much sense without 2/2, it >> probably would have been better to do these as a single patch. > > Would you like me to resubmit it as a single patch, or are you > applying them as is? > >> And

Re: [PATCH 2/2] pretty: add %D format specifier

2014-09-11 Thread Harry Jeffery
On 11/09/14 17:56, Junio C Hamano wrote: > Because patch 1/2 alone does not make much sense without 2/2, it > probably would have been better to do these as a single patch. Would you like me to resubmit it as a single patch, or are you applying them as is? > And of course a few additional test

Re: [PATCH 2/2] pretty: add %D format specifier

2014-09-11 Thread Junio C Hamano
Because patch 1/2 alone does not make much sense without 2/2, it probably would have been better to do these as a single patch. And of course a few additional tests to t4205 would not hurt ;-) Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to m

[PATCH 2/2] pretty: add %D format specifier

2014-09-10 Thread Harry Jeffery
%d prints decorations wrapped by " (" and ")". %D provides the same output without the parenthesis, making " (%D)" and "%d" equivalent to one another. Signed-off-by: Harry Jeffery --- Documentation/pretty-formats.txt | 6 -- pretty.c | 4 2 files changed, 8 inse