From c5e2085629069f44adbc6d193824484a91320843 Mon Sep 17 00:00:00 2001 From: Greg Clayton Date: Mon, 10 Dec 2012 19:43:43 +0000 Subject: [PATCH] Fix the OptionValueFileSpec option value to correctly get the file path when trailing spaces are on the path. The "settings set" command uses the OptionValueFileSpec class to set file paths and if extra spaces are at the end it will include those in the paths. Now we chop up the value send to to OptionValueFileSpec::SetValueFromCString(...) function with "lldb_private::Args" and give an appropriate error if more than one path is used. It also allows for quotes to be used when specifying the path. llvm-svn: 169753 --- lldb/source/Interpreter/OptionValueFileSpec.cpp | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/lldb/source/Interpreter/OptionValueFileSpec.cpp b/lldb/source/Interpreter/OptionValueFileSpec.cpp index cb1d00c..96bc0e0 100644 --- a/lldb/source/Interpreter/OptionValueFileSpec.cpp +++ b/lldb/source/Interpreter/OptionValueFileSpec.cpp @@ -94,8 +94,17 @@ OptionValueFileSpec::SetValueFromCString (const char *value_cstr, case eVarSetOperationAssign: if (value_cstr && value_cstr[0]) { - m_value_was_set = true; - m_current_value.SetFile(value_cstr, value_cstr[0] == '~'); + Args args(value_cstr); + if (args.GetArgumentCount() == 1) + { + const char *path = args.GetArgumentAtIndex(0); + m_value_was_set = true; + m_current_value.SetFile(path, true); + } + else + { + error.SetErrorString("please supply a single path argument for this file or quote the path if it contains spaces"); + } } else { -- 2.7.4