[Lldb-commits] [PATCH] Allow multiple simultaneous connections to lldb-platform

[Tamas Berghammer]

Looks good but please consider my inline comment


================
Comment at: include/lldb/Host/Socket.h:64
@@ -60,1 +63,3 @@
+        Predicate<uint16_t>* predicate,
+        int backlog = 5);
     static Error TcpConnect(llvm::StringRef host_and_port, bool child_processes_inherit, Socket *&socket);
----------------
I think you should leave the default value to be 1 unless you are sure nobody is depending on the original behavior.

http://reviews.llvm.org/D8696

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/