[Lldb-commits] [PATCH] D14085: Make Socket to support plugin interface

Oleksiy Vyalov via lldb-commits lldb-commits at lists.llvm.org
Mon Oct 26 12:07:06 PDT 2015


ovyalov added a comment.

In http://reviews.llvm.org/D14085#275442, @zturner wrote:

> I'm not sure I agree with this change.  The interface to programming with socket is by definition a property of the Host operating system.  It seems to me like Host was already the correct place for this code.


Base Socket class which is utilized by users remains in Host interface - moving implementation classes into Plugins somewhat simplifies creation/registration process with PluginManager (instead of if (scheme=="tcp"){} else if (scheme=="unix"){}..).


http://reviews.llvm.org/D14085





More information about the lldb-commits mailing list