• 
      

    Fix the displayed line numbers in e-mails with filtered interdiffs.

    Review Request #8246 — Created June 16, 2016 and submitted — Latest diff uploaded

    Information

    Review Board
    release-2.5.x
    6de893c...

    Reviewers

    The new code for calculating displayed line numbers in e-mails didn't
    take into account that filtered interdiffs could result in equal chunks
    without line numbers. It's been updated to check whether there are line
    numbers in the appropriate side of the chunk before considering that
    side valid.
    
    Unit tests have been updated for these conditions.

    Unit tests pass. Before, the new tests were failing with the same error
    we saw in production with the same conditions.