Fix showing indentation changes in interdiffs.

Review Request #12068 — Created Feb. 20, 2022 and submitted — Latest diff uploaded

Information

Review Board
release-4.0.x

Reviewers

When viewing an interdiff, indentation changes wouldn't show up. If the
only changes made to a file were indentation-related, the file would
display only the last few lines of context, along with a banner stating
"This file contains only whitespace changes."

This was due to a strict check against an opcode tag of equal,
which should have allowed for filtered-equal as well.

We now check both, and split off new equal ranges for anything
containing indentation changes, ensuring they're highlighted. All other
filtered-equal ranges will remain.

Viewed an interdiff where nothing but indentation changes and trailing
whitespace had changed. Saw that those chunks were visible in the diff
viewer.

Commits

Files

    Loading...