Page MenuHomePhabricator

Fix "left"/"right" changeset ID selection for synthetic deletions
ClosedPublic

Authored by epriestley on May 17 2017, 3:27 PM.
Tags
None
Referenced Files
F18739808: D17929.diff
Wed, Oct 1, 11:01 PM
F18730552: D17929.id.diff
Tue, Sep 30, 1:18 PM
F18717189: D17929.id43123.diff
Mon, Sep 29, 1:18 PM
F18679209: D17929.id.diff
Sep 26 2025, 12:34 AM
F18635637: D17929.id.diff
Sep 17 2025, 2:21 AM
F18627421: D17929.id43123.diff
Sep 16 2025, 3:58 AM
F18621778: D17929.id43123.diff
Sep 15 2025, 9:25 AM
F18621776: D17929.id43123.diff
Sep 15 2025, 9:25 AM
Subscribers
None

Details

Summary

Fixes T8323. See that task for a description.

We were using nonempty(), but that rule doesn't cover synthetic deletions (file present in an earlier diff, but no longer present in the later diff).

Test Plan

Followed the steps in T8323, got a clean comment.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.May 17 2017, 3:31 PM
This revision was automatically updated to reflect the committed changes.