Summary: Diffs containing only indentation changes causes the diff viewer
to look empty
Labels: -Priority-Medium Priority-Low Component-DiffViewer
Comment #1 on issue 1511 by chipx86: Diffs containing only indentation
changes causes the diff viewer to look empty
Okay, I've investigated this a little and it can basically be reproduced
with a diff
--- filename.foo (...)
+++ filename.foo (...)
@@ ... @@
Notice how the only thing that is changing is the indentation. To Review
really means nothing has changed. We don't show leading whitespace changes.
internally, it's finding that nothing has changed with the code and when we
render this and see no changed chunks, we consider this to be a file in the
without any changes and we remove it from the list. This is important for
which is mostly why the logic is there.
So, this particular diff in question isn't doing anything but changing
and Review Board is therefore not seeing that anything is really up for
We probably should be a little more graceful about this, but I will point
this is the first time we've seen this sort of an issue where a diff
nothing but indentation changes has gone up and confused people. We can
look at may
be special-casing non-interdiffs and displaying some indication that the
Given the rarity of the case, though, it's not a huge priority at the
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
You received this message because you are subscribed to the Google Groups
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to
For more options, visit this group at