Re: [tig] [PATCHv2 3/3] log: Colour the diff stat
To
Jonas Fonseca
Cc
git
From
Kumar Appaiah
See Also
Prev Ref 1 Ref 2 Ref 3
Date
2014-04-17 01:04:54 UTC

On Wed, Apr 16, 2014 at 08:44:41PM -0400, Jonas Fonseca wrote:
> On Sun, Apr 13, 2014 at 5:54 PM, Kumar Appaiah
> <a.kumar@alumni.iitm.ac.in> wrote:
> >
> > This commit adds custom log_read and log_draw functions that utilize
> > the diff stat drawing functions from the diff module. The absence of
> > the triple hyphen separator prevents direct usage of the diff drawing
> > functions directly.
> 
> See my comments below.

Hi Jonas.

> > +static bool
> > +log_draw(struct view *view, struct line *line, unsigned int lineno)
> > +{
> > +       char *text = line->data;
> > +       enum line_type type = line->type;
> > +
> 
> This is missing a call to draw_lineno(...)

Noted.

> > +       if (type == LINE_DIFF_STAT) {
> > +               diff_common_draw_diff_stat(view, &type, &text);
> > +               draw_text(view, type, text);
> 
> I had to #include "tig/draw.h" for this to compile.

I'll take care of this.

I'll send you a pull request eventually. You can handle it after your
refactor is complete.

Thanks!

Kumar
-- 
Kumar Appaiah