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.

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:

  PythonDictionary 
dictionary(m_interpreter->OSPlugin_RegisterInfo(m_python_object_sp));

Will need to become:

  StructuredData::Dictionary 
dictionary(m_interpreter->OSPlugin_RegisterInfo(m_python_object_sp));

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.


http://reviews.llvm.org/D7957

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

Reply via email to