[Lldb-commits] [lldb] r176719 - OptionValueFileSpec shouldn't be doing argument

Sean Callanan scallanan at apple.com
Fri Mar 8 12:59:47 PST 2013


Author: spyffe
Date: Fri Mar  8 14:59:47 2013
New Revision: 176719

URL: http://llvm.org/viewvc/llvm-project?rev=176719&view=rev
Log:
OptionValueFileSpec shouldn't be doing argument
parsing on the file name it gets.  That confuses
it if there are spaces in the file name.

<rdar://problem/13380392>

Modified:
    lldb/trunk/source/Interpreter/OptionValueFileSpec.cpp

Modified: lldb/trunk/source/Interpreter/OptionValueFileSpec.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Interpreter/OptionValueFileSpec.cpp?rev=176719&r1=176718&r2=176719&view=diff
==============================================================================
--- lldb/trunk/source/Interpreter/OptionValueFileSpec.cpp (original)
+++ lldb/trunk/source/Interpreter/OptionValueFileSpec.cpp Fri Mar  8 14:59:47 2013
@@ -94,17 +94,8 @@ OptionValueFileSpec::SetValueFromCString
     case eVarSetOperationAssign:
         if (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");
-            }
+            m_value_was_set = true;
+            m_current_value.SetFile(value_cstr, true);
         }
         else
         {





More information about the lldb-commits mailing list