[Lldb-commits] [PATCH] D13754: Split Socket class into Tcp/Udp/DomainSocket subclasses.

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Fri Oct 16 01:11:15 PDT 2015


labath added inline comments.

================
Comment at: source/Host/posix/DomainSocket.cpp:74
@@ +73,3 @@
+    FileSystem::Unlink(FileSpec{name, true});
+
+    Error error;
----------------
If we're going to use unique names, then this won't be necessary (and I would much rather see a random error opening a socket than a random file disappearing). BTW, have you considered using abstract sockets for the lldb-server use case? Albeit linux-specific, I find them much nicer, as they have no connection to the file system whatsoever.


http://reviews.llvm.org/D13754





More information about the lldb-commits mailing list