[Lldb-commits] [PATCH] D95711: [lldb/Interpreter] Add ScriptInterpreter Wrapper for ScriptedProcess
Med Ismail Bennani via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Feb 12 14:30:53 PST 2021
mib updated this revision to Diff 323481.
mib marked 3 inline comments as done.
mib added a comment.
Address @JDevlieghere comments:
- Fix namespace closing comment in `SBData.h`
- Remove `ScriptInterpreter` overload by making the `ScriptedProcessInterface` a default argument.
- Replace inline comments with error messages.
Also, since `TestScriptInterpreterPython` inherits from `ScriptInterpreterPythonImpl` it needed some symbol definitions.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95711/new/
https://reviews.llvm.org/D95711
Files:
lldb/bindings/python/python-wrapper.swig
lldb/include/lldb/API/SBData.h
lldb/include/lldb/API/SBThreadPlan.h
lldb/include/lldb/Interpreter/ScriptInterpreter.h
lldb/include/lldb/Interpreter/ScriptedProcessInterface.h
lldb/include/lldb/lldb-forward.h
lldb/source/Interpreter/CMakeLists.txt
lldb/source/Interpreter/ScriptInterpreter.cpp
lldb/source/Interpreter/ScriptedProcessInterface.cpp
lldb/source/Plugins/ScriptInterpreter/Python/CMakeLists.txt
lldb/source/Plugins/ScriptInterpreter/Python/SWIGPythonBridge.cpp
lldb/source/Plugins/ScriptInterpreter/Python/SWIGPythonBridge.h
lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp
lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.h
lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPythonImpl.h
lldb/source/Plugins/ScriptInterpreter/Python/ScriptedProcessPythonInterface.cpp
lldb/source/Plugins/ScriptInterpreter/Python/ScriptedProcessPythonInterface.h
lldb/unittests/ScriptInterpreter/Python/CMakeLists.txt
lldb/unittests/ScriptInterpreter/Python/PythonTestSuite.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D95711.323481.patch
Type: text/x-patch
Size: 33115 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20210212/8cbfad46/attachment-0001.bin>
More information about the lldb-commits
mailing list