[lldb-vscode] correctly handle multiple sourceMap entries

[rmaz]

Summary

lldb-vscode concatenates a string of sourceMap entries
specified in the config, but fails to put a space between
each entry, which causes the settings command to fail.
This patch adds a space between each mapping.

[xiaobai]

Looks fine to me. Any objections @clayborg?

[rmaz]

Thanks for the review, how can I get this landed now?

[xiaobai]

@rmaz: I can land this for you if you would like me to.

[rmaz]

That would be good, thanks

[labath]

A test case?

lldb/trunk/tools/lldb-vscode/lldb-vscode.cpp

for (const auto &value : *sourceMap) {
       if (mapping == nullptr || mapping->size() != 2 ||
           (*mapping)[0].kind() != llvm::json::Value::String ||
           (*mapping)[1].kind() != llvm::json::Value::String) {
         g_vsc.SendOutput(OutputType::Console, llvm::StringRef(sourceMapHelp));
         return;
       }
       auto mapFrom = GetAsString((*mapping)[0]);
       auto mapTo = GetAsString((*mapping)[1]);
-      strm << "\"" << mapFrom << "\" \"" << mapTo << "\"";
+      strm << "\"" << mapFrom << "\" \"" << mapTo << "\" ";
     }
   } else {
     if (ObjectContainsKey(arguments, "sourceMap")) {
       g_vsc.SendOutput(OutputType::Console, llvm::StringRef(sourceMapHelp));
       return;
     }
     if (sourcePath.empty())
       return;
static std::map<std::string, RequestCallback> g_request_handlers = {
       REQUEST_CALLBACK(threads),
       REQUEST_CALLBACK(variables),
       // Testing requests
       REQUEST_CALLBACK(_testGetTargetBreakpoints),
   };
 #undef REQUEST_CALLBACK
   return g_request_handlers;
 }
-  
 } // anonymous namespace
 
 int main(int argc, char *argv[]) {
 
   // Initialize LLDB first before we do anything.
   lldb::SBDebugger::Initialize();
 
   if (argc == 2) {