[Lldb-commits] [PATCH] Remove lldb_private library implicit dependencies on python

Zachary Turner zturner at google.com
Tue Jun 16 15:02:38 PDT 2015


The goal of ScriptInterpreterFactory was to remove the dependency from
CommandInterpreter -> ScriptInterpreterPython.  If it can be done with a
plugin as you suggest, then that should work fine.

In any case, I'm OOO for another 5 weeks, so it won't happen until then.

On Tue, Jun 16, 2015 at 5:47 AM Greg Clayton <clayborg at gmail.com> wrote:

> Why do we need a ScriptInterpreterFactory class and then pass around a
> unique pointer to it? Why don't we just add the script interpreters to
> PluginManager.h and PluginManager.cpp and have all script interpreter
> languages add themselves as supported script interpreter plug-ins with a
> create method. Seems more consistent with all of the other plug-in based
> things we support.
>
>
> http://reviews.llvm.org/D10189
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150616/27c802d2/attachment.html>


More information about the lldb-commits mailing list