<div dir="ltr">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.<div><br></div><div>In any case, I'm OOO for another 5 weeks, so it won't happen until then.</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jun 16, 2015 at 5:47 AM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">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.<br>
<br>
<br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10189&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=YsKvQ1lBC1e2Aa4iJzXUAmFmBYiadqZC8aML4h9VkGU&s=9O15QnwRQLtIyenaVB5f0ZtSO91h48VrqgcmaGYW2Ic&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10189</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_settings_panel_emailpreferences_&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=YsKvQ1lBC1e2Aa4iJzXUAmFmBYiadqZC8aML4h9VkGU&s=2Km5eYxLIEKzBQQq8fWrOcsBbWdIJtGUNif0zKsfD-w&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div>