Subject: Re: [PATCH v2 3/8] gitweb: Extract print_sidebyside_diff_lines()

From: Jakub Narebski <[email-masked]>
Date: Fri, 30 Mar 2012 14:37:38 +0100

On Thu, 29 Mar 2012, Michał Kiedrowicz wrote:
> Jakub Narebski <[email-masked]> wrote:
> > On Fri, 23 Mar 2012, Michał Kiedrowicz wrote:

> > > This is a preparation patch for diff refinement highlightning.  It should
> > > not change the gitweb output, but it slightly changes its behavior.
> > > Before this commit, context is printed on the class change. Now, it'it
> > > printed just before printing added and removed lines.
> > 
> >                                                       , and at the end
> >   of chunk.

Please don't forget to add that information.  I was wondering about this,
and checked code that it works all right.

> > IMVHO such change is irrelevant.

This refers to "but it slightly changes its behavior", which might be
not clear from the position of this sentence.

> > Acked-by: Jakub Narębski <[email-masked]>
> 
> Thanks.

-- 
Jakub Narebski
Poland