Reimplemented the code that backed the "settings" in lldb. There were many issues with the previous implementation:

- no setting auto completion
- very manual and error prone way of getting/setting variables
- tons of code duplication
- useless instance names for processes, threads

Now settings can easily be defined like option values. The new settings makes use of the "OptionValue" classes so we can re-use the option value code that we use to set settings in command options. No more instances, just "does the right thing".

llvm-svn: 162366

1 files changed, 1 insertions, 1 deletions

diff --git a/lldb/source/Core/FileSpecList.cpp b/lldb/source/Core/FileSpecList.cpp
index ba2e95ec1bb..a24494d16e4 100644
--- a/lldb/source/Core/FileSpecList.cpp
+++ b/lldb/source/Core/FileSpecList.cpp
@@ -94,7 +94,7 @@ FileSpecList::Dump(Stream *s, const char *separator_cstr) const
     for (pos = m_files.begin(); pos != end; ++pos)
     {
         pos->Dump(s);
-        if (separator_cstr)
+        if (separator_cstr && ((pos + 1) != end))
             s->PutCString(separator_cstr);
     }
 }