HomePhabricator

Various minor improvements to the `xsprintf` functions

Description

Various minor improvements to the xsprintf functions

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.

Reviewers: Blessed Reviewers, epriestley

Reviewed By: Blessed Reviewers, epriestley

Subscribers: epriestley, Korvin

Differential Revision: https://secure.phabricator.com/D9788

Details

Provenance
joshuaspenceAuthored on
joshuaspencePushed on Jul 1 2014, 2:07 PM
Reviewer
Blessed Reviewers
Differential Revision
D9788: Various minor improvements to the `xsprintf` functions
Parents
rPHUb2903bea0271: Test cases should extend from `PhutilTestCase`
Branches
Unknown
Tags
Unknown

Event Timeline