HomePhorge

Implement rough content-aware inline adjustment rules for ghosts

Description

Implement rough content-aware inline adjustment rules for ghosts

Summary:
Ref T7447. Fixes T7600. This likely needs significant adjustment, but implements content-aware comment porting for line changes.

Specifically, this moves lines around to adjust their position considering added and removed lines between the diffs and across rebases.

It does not try to do any actual content (line against line) matching.

Test Plan:

  • Unit tests.
  • Poking around in the web UI seems to generate mostly reasonable-ish results?
  • This may be a huge step backward in some cases that I just haven't hit.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: yelirekim, epriestley

Maniphest Tasks: T7600, T7447

Differential Revision: https://secure.phabricator.com/D12741

Details

Provenance
epriestleyAuthored on May 7 2015, 2:09 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP524aee03dca0: Conpherence - make sure real time updates still work if you switch threads
Branches
Unknown
Tags
Unknown

Event Timeline