I will update the code tonight.

On 29 jan. 2014, at 14:20, Dmitriy Zaporozhets <notifications@github.com> wrote:

> In app/views/projects/commits/_parallel_view.html.haml:
> 
> > @@ -7,9 +7,8 @@
> >      - deleted_lines[line_old] = { line_code: line_code, type: type, line: line }
> >    - elsif type == "new"
> >      - added_lines[line_new]   = { line_code: line_code, type: type, line: line }
> > -
> > -- max_length = old_file.sloc + added_lines.length if old_file
> > -- max_length ||= file.sloc
> > +- max_length = [(file.loc - 1), (old_file.loc - 1 + added_lines.length), 0].max if old_file
> View is not the best place for application logic. 
> Please move such logic to separate method/class.
> 
> —
> Reply to this email directly or view it on GitHub.
>