Page MenuHomePhabricator

Minor `bin/aphlict` cleanup
ClosedPublic

Authored by epriestley on Apr 13 2016, 3:52 PM.
Tags
None
Referenced Files
F19514348: D15698.id37822.diff
Wed, Jan 14, 1:05 AM
F19510139: D15698.id37822.diff
Sun, Jan 11, 5:10 AM
F19470456: D15698.id.diff
Mon, Jan 5, 3:58 PM
F19470374: D15698.id.diff
Mon, Jan 5, 2:37 PM
F19468956: D15698.id.diff
Sun, Jan 4, 1:14 PM
F19468948: D15698.diff
Sun, Jan 4, 1:08 PM
F18831709: D15698.id.diff
Oct 25 2025, 2:45 PM
F18827937: D15698.diff
Oct 24 2025, 1:12 PM
Subscribers
None

Details

Summary

Ref T10697. This just improves a couple of minor bin/aphlict things: make argument parsing more explicit/consistent, consolidate a little bit of duplicated code.

Test Plan

Ran all bin/aphlict commands.

Diff Detail

Repository
rP Phabricator
Branch
aphlict1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11690
Build 14636: Run Core Tests
Build 14635: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Minor `bin/aphlict` cleanup.
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.Apr 13 2016, 5:34 PM
This revision was automatically updated to reflect the committed changes.