Page MenuHomePhabricator

Don't explicitly name abstract base classes
ClosedPublic

Authored by joshuaspence on Jul 20 2014, 10:40 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Aug 17, 10:27 AM
Unknown Object (File)
Thu, Jul 28, 9:42 AM
Unknown Object (File)
Jul 13 2022, 4:23 AM
Unknown Object (File)
Jul 10 2022, 11:03 PM
Unknown Object (File)
Jun 30 2022, 9:54 AM
Unknown Object (File)
Jun 30 2022, 9:53 AM
Unknown Object (File)
Jun 30 2022, 9:53 AM
Unknown Object (File)
Jun 30 2022, 9:53 AM
Subscribers

Details

Summary

Ref T5655. It is superfluous to include "base" in the name of an abstract base class. Furthermore, it is not done consistently within the code base.

Test Plan

Ran arc unit.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Don't explicitly name abstract base classes.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jul 21 2014, 1:26 PM
joshuaspence updated this revision to Diff 24028.

Closed by commit rP37106c1b3178 (authored by @joshuaspence).