Page MenuHomePhabricator

Improve handle semantics with HandlePool / HandleList
ClosedPublic

Authored by epriestley on Mar 30 2015, 1:27 AM.
Tags
None
Referenced Files
F15490166: D12202.id29352.diff
Fri, Apr 11, 2:54 PM
F15478015: D12202.id29325.diff
Mon, Apr 7, 7:39 PM
F15474585: D12202.id.diff
Sun, Apr 6, 9:55 AM
F15471429: D12202.diff
Sat, Apr 5, 4:57 AM
F15435035: D12202.id29325.diff
Tue, Mar 25, 6:34 AM
F15384372: D12202.id29352.diff
Fri, Mar 14, 7:41 PM
F15384097: D12202.id29325.diff
Fri, Mar 14, 6:46 PM
F15380193: D12202.id.diff
Fri, Mar 14, 1:56 AM
Subscribers
Tokens
"Grey Medal" token, awarded by chad.

Details

Summary

Ref T7689, which discusses some of the motivation here. Briefly, these methods are awkward:

  • Controller->loadHandles()
  • Controller->loadViewerHandles()
  • Controller->renderHandlesForPHIDs()

This moves us toward better semantics, less awkwardness, and a more reasonable attack on T7688 which won't double-fetch a bunch of data.

Test Plan
  • Added unit tests.
  • Converted one controller to the new stuff.
    • Viewed countdown lists, saw handles render.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Improve handle semantics with HandlePool / HandleList.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 30 2015, 7:54 PM
This revision was automatically updated to reflect the committed changes.