HomePhabricator

Better change/add/delete distinction for DiffOps

Authored by MaxSem on Jun 29 2016, 5:49 AM.

Description

Better change/add/delete distinction for DiffOps

Improve handling of change/add/delete ops in diff engine so Change 319866 gets a chance to run.

This uses existing diff code to compare the LHS and RHS of every DiffOp::change, and convert it to an add+delete pair if they are too dissimilar (current threshold is 25% identical chars). This functionality is already in Change 319866. This patch contains only the change vs add+del stuff without moved-paragraph-detection, for easier review.

Bug: T150740
Bug: T149674
Change-Id: If0952aa53e472c23237890ee6d6997ac9d6d7c0e

Details

Committed
MaxSemSep 5 2017, 6:51 PM
Parents
rMWDJc69d0502e222: Merge "Fuzz test"
Branches
Unknown
Tags
Unknown
References
refs/changes/82/356582/24
ChangeId
If0952aa53e472c23237890ee6d6997ac9d6d7c0e