<div dir="ltr">I think this part needs to be done as a separate patch.  Makes it easier to review and understand the impact of.  So I will do this part first (basically delete ScriptInterpreterObject and move everything to StructuredData) and commit that, then rebase this on top of that and go from there.<br><br><div class="gmail_quote">On Thu, Mar 5, 2015 at 2:37 PM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">So seems like part of the work required for this patch to work is to change any references in code from PythonList, PythonString, PythonInteger, and PythonDictionary to use classes from lldb/Core/StructuredData.h.<br>
<br>
The means we probably need to have OperatingSystemPython rely only upon virtual functions in ScriptInterpreter and have those functions return stuff from StructuredData instead of python variants:<br>
<br>
  PythonDictionary dictionary(m_interpreter-><u></u>OSPlugin_RegisterInfo(m_<u></u>python_object_sp));<br>
<br>
Will need to become:<br>
<br>
  StructuredData::Dictionary dictionary(m_interpreter-><u></u>OSPlugin_RegisterInfo(m_<u></u>python_object_sp));<br>
<br>
We then need to modify the Python callbacks that return PythonList, PythonString, PythonInteger, and PythonDictionary objects, to convert them into StructuredData::Array, StructuredData::String, StructuredData::Integer and StructuredData::Dictionary respectively.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D7957" target="_blank">http://reviews.llvm.org/D7957</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote></div></div>