Re: [PATCH v2 02/12] pretty: share code between format_decoration and show_decorations

2013-04-12 Thread Duy Nguyen
On Fri, Apr 5, 2013 at 6:57 PM, Jakub Narębski wrote: >>> +void format_decoration(struct strbuf *sb, >>> + const struct commit *commit, >>> + int use_color); >> >> I think you can fit these on a single line, especially if you drop >> the unused variable names (t

Re: [PATCH v2 02/12] pretty: share code between format_decoration and show_decorations

2013-04-12 Thread Duy Nguyen
Sorry for this late reply. I've been quite busy lately.. On Tue, Apr 2, 2013 at 4:53 AM, Junio C Hamano wrote: >> -void show_decorations(struct rev_info *opt, struct commit *commit) >> +void format_decoration(struct strbuf *sb, >> +const struct commit *commit, >> +

Re: [PATCH v2 02/12] pretty: share code between format_decoration and show_decorations

2013-04-05 Thread Jakub Narębski
Junio C Hamano wrote: > Nguyễn Thái Ngọc Duy writes: >> diff --git a/log-tree.h b/log-tree.h >> index 9140f48..e6a2da5 100644 >> --- a/log-tree.h >> +++ b/log-tree.h >> @@ -13,6 +13,9 @@ int log_tree_diff_flush(struct rev_info *); >> int log_tree_commit(struct rev_info *, struct commit *); >>

Re: [PATCH v2 02/12] pretty: share code between format_decoration and show_decorations

2013-04-01 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy writes: > This also adds color support to format_decoration() > > Signed-off-by: Nguyễn Thái Ngọc Duy > --- > log-tree.c | 60 > +--- > log-tree.h | 3 ++ > pretty.c |

[PATCH v2 02/12] pretty: share code between format_decoration and show_decorations

2013-03-30 Thread Nguyễn Thái Ngọc Duy
This also adds color support to format_decoration() Signed-off-by: Nguyễn Thái Ngọc Duy --- log-tree.c | 60 +--- log-tree.h | 3 ++ pretty.c | 19 + t/t4207-log-decoration-colors