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 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
