Support marking files as deleted in the diff viewer.

Review Request #1685 — Created June 26, 2010 and submitted — Latest diff uploaded

Information

Review Board

Reviewers

Support marking files as deleted in the diff viewer.

We've always treated deleted files as just another file change. A file with
a bunch of removes. However, this isn't as clear, and it's often just extra
noise. In the case of CVS, it actually doesn't work, because the files
referenced in the diff don't contain enough data to locate the source file.

Diff parsers can now flag a file as being deleted, which will prevent
any file existence checks and instead indicate in the diff viewer that the
file was deleted. This is similar in appearance to the binary indicator.

Right now, only CVS supports this, but support can easily be added to other
SCMTools later.
Verified that I could take a CVS diff with a deleted file and view it in the diff viewer.
Previously, I could not, and it was failing with an error.
    Loading...