• 
      

    Suppress errors when trying to generate stats for empty diffs.

    Review Request #11796 — Created Aug. 18, 2021 and submitted — Latest diff uploaded

    Information

    DiffX
    release-1.0.x

    Reviewers

    There are cases where a diffx diff section will have empty diff
    content--for example, when a file is renamed but doesn't have any
    changes to content. The diff stats generation will do the right thing if
    a file is added without specifying diff= in the constructor at all,
    but was spewing errors with diff=b''. This change updates things to
    check for falsiness rather than explicitly checking against None.

    • Ran with some code that was creating empty (but not None) diffs. Saw no
      more errors.
    • Ran unit tests.

    Commits

    Files