Page MenuHomePhabricator

Remove buildable handle / container handle logic form Harbormaster buildable queries
ClosedPublic

Authored by epriestley on Feb 26 2016, 8:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 9, 8:01 AM
Unknown Object (File)
Tue, Mar 5, 9:25 AM
Unknown Object (File)
Tue, Mar 5, 9:25 AM
Unknown Object (File)
Tue, Mar 5, 9:24 AM
Unknown Object (File)
Tue, Mar 5, 7:16 AM
Unknown Object (File)
Feb 10 2024, 10:06 PM
Unknown Object (File)
Feb 10 2024, 10:04 PM
Unknown Object (File)
Feb 10 2024, 9:58 PM
Subscribers
None

Details

Summary

Ref T10457. We currently have these weird, out-of-place methods on Harbormaster queries that just load handles. These were written before HandlePool, and HandlePool is now more convenient, simpler, and more efficient.

Drop this stuff in favor of using handle pools off $viewer.

Test Plan

Looked at buildable list, looked at buildable detail, grepped for removed methods.

Diff Detail

Repository
rP Phabricator
Branch
dry3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 10863
Build 13393: Run Core Tests
Build 13392: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Remove buildable handle / container handle logic form Harbormaster buildable queries.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Feb 26 2016, 8:56 PM
This revision was automatically updated to reflect the committed changes.