[Lldb-commits] [PATCH] D13970: Add support for abstract domain sockets.

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Thu Oct 22 14:46:51 PDT 2015


clayborg added a comment.

In http://reviews.llvm.org/D13970#273267, @ovyalov wrote:

> In http://reviews.llvm.org/D13970#273217, @clayborg wrote:
>
> > Looks good. At some point we should add these to the PluginMananger and have new instances found by having them register URL prefixes which match up with CreateInstance methods that are given the URL.
>
>
> Thank you for the idea - I was planning to have a kind of URL-based factory inside lldb-server/Acceptor and PluginManager looks as a way to go.
>  If we're going to make Socket class a plugin should we then move all implementations into source/Plugins/Socket folder?


Yes.


http://reviews.llvm.org/D13970





More information about the lldb-commits mailing list