Page MenuHomePhabricator

Clean up various pieces of dead/obsolete Differential code
ClosedPublic

Authored by epriestley on Mar 9 2014, 8:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 6, 10:36 AM
Unknown Object (File)
Mon, Aug 26, 3:15 PM
Unknown Object (File)
Thu, Aug 15, 8:01 PM
Unknown Object (File)
Mon, Aug 12, 10:08 AM
Unknown Object (File)
Mon, Aug 12, 10:04 AM
Unknown Object (File)
Mon, Aug 12, 10:03 AM
Unknown Object (File)
Mon, Aug 12, 10:00 AM
Unknown Object (File)
Aug 7 2024, 10:11 AM
Subscribers

Details

Summary

Ref T2222.

  • Removes DifferentialTasksAttacher, which has had no callsites for a very long time.
  • Moves differential.getrevisioncomments off DifferentialCommentQuery.
  • Moves Releeph churn field off DifferentialCommentQuery.
  • Removes dead code in DifferentialRevisionViewController.
  • Removes DifferentialException (no references).
  • Removes DifferentialRevision->loadComments() (no callsites).
  • Removes DifferentialRevision->loadReviewedBy() (all callsites updated).
  • Removes DifferentialCommentQuery (all callsites updated).
Test Plan

Mostly a lot of grep.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 10 2014, 6:48 PM
epriestley updated this revision to Diff 20129.

Closed by commit rP592591e71538 (authored by @epriestley).