Short-circuit diff patching for moved files with no changes.
Review Request #6655 — Created Dec. 1, 2014 and submitted
The comment for
DiffChunkGenerator.get_chunks()indicates that moved files
with no changes should be returned as an empty list without actually fetching
source files or applying patches, but this wasn't the case. This change fixes
the conditional to return the empty list.
- Tested a moved file with no changes. Saw that it displayed correctly without
ever calling out to 'patch' - Tested a moved+edited file. Saw that it displayed correctly.
- Ran unit tests.
Tool: Pyflakes Processed Files: reviewboard/diffviewer/chunk_generator.py Tool: PEP8 Style Checker Processed Files: reviewboard/diffviewer/chunk_generator.py