Page MenuHomePhabricator

Various minor improvements to the `xsprintf` functions
ClosedPublic

Authored by joshuaspence on Jun 30 2014, 10:19 PM.
Tags
None
Referenced Files
F15335604: D9788.id23499.diff
Sat, Mar 8, 4:49 PM
Unknown Object (File)
Tue, Feb 25, 5:05 PM
Unknown Object (File)
Mon, Feb 24, 3:29 PM
Unknown Object (File)
Mon, Feb 24, 3:28 PM
Unknown Object (File)
Mon, Feb 24, 3:06 PM
Unknown Object (File)
Mon, Feb 24, 3:02 PM
Unknown Object (File)
Mon, Feb 24, 3:01 PM
Unknown Object (File)
Mon, Feb 24, 2:29 PM
Subscribers

Details

Summary
  • Renamed Phutil*sprintfTestCase classes to use proper camel case.
  • Added some type hints for function parameters, where appropriate.
  • Use more specific (SPL) Exception types, where appropriate.
  • Change ## to \` in comments.
  • Wrap functions in @{function:...} when they appear in comments.
Test Plan

arc unit still works.

Diff Detail

Repository
rPHU libphutil
Branch
xsprintf
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1473
Build 1473: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Various minor improvements to the `xsprintf` functions.
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 1 2014, 1:19 PM