Page MenuHomePhabricator

Add "%R" to csprintf(), for printing more readable commands
ClosedPublic

Authored by epriestley on Jan 29 2014, 10:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 9:53 PM
Unknown Object (File)
Tue, Apr 23, 11:24 PM
Unknown Object (File)
Tue, Apr 23, 9:38 PM
Unknown Object (File)
Tue, Apr 23, 9:38 PM
Unknown Object (File)
Tue, Apr 23, 9:38 PM
Unknown Object (File)
Tue, Apr 23, 9:10 AM
Unknown Object (File)
Sun, Apr 21, 4:08 PM
Unknown Object (File)
Sat, Apr 20, 2:45 PM
Subscribers

Details

Summary

Ref T4175. The clone commands we print have quoted arguments, but usually don't need them. Add "%R", which is like "%s" but only adds quotes if they're required. This produces slightly more readable commands in the few cases we're generating commands for users to copy/paste.

Test Plan

Added unit tests. Looked up shell metacharacters to double check my definition.

Diff Detail

Repository
rPHU libphutil
Branch
readesc
Lint
Lint Passed
Unit
Tests Passed