From d420c31e107cd932582a4a8add0ce2676e4b52da Mon Sep 17 00:00:00 2001 From: "evan@chromium.org" Date: Fri, 30 Jul 2010 02:14:22 +0000 Subject: Add an AppendSwitchASCII to CommandLine, and convert a test to it. I'm removing all the AppendSwitchWithValue() users due to wstrings, and this is one caller. Since fixing this one caller requires touching many files, I thought I'd isolate this change from the other WithValue->ASCII conversions. Review URL: http://codereview.chromium.org/2878065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54257 0039d316-1c4b-4281-b951-d872f2087c98 --- base/command_line.h | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'base/command_line.h') diff --git a/base/command_line.h b/base/command_line.h index 89df392..ae05021 100644 --- a/base/command_line.h +++ b/base/command_line.h @@ -162,14 +162,17 @@ class CommandLine { void AppendSwitchPath(const std::string& switch_string, const FilePath& path); void AppendSwitchNative(const std::string& switch_string, const StringType& value); + void AppendSwitchASCII(const std::string& switch_string, + const std::string& value); // Append a switch and value to the command line. // TODO(evanm): remove all AppendSwitchWithValue() instances. - // TODO(evanm): add an *ASCII() version. void AppendSwitchWithValue(const std::string& switch_string, const std::wstring& value_string); void AppendSwitchWithValue(const std::string& switch_string, - const std::string& value_string); + const std::string& value_string) { + AppendSwitchASCII(switch_string, value_string); + } // Append a loose value to the command line. void AppendLooseValue(const std::wstring& value); -- cgit v1.1