[Lldb-commits] [PATCH] D143236: [lldb] Add a way to get a scripted process implementation from the SBAPI
Jim Ingham via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Feb 3 10:23:13 PST 2023
jingham requested changes to this revision.
jingham added inline comments.
This revision now requires changes to proceed.
================
Comment at: lldb/bindings/python/python-typemaps.swig:57
+%typemap(out) void* {
+ $result = nullptr;
----------------
Won't this make every out void * claim to be a Python Object in the SWIG-ed output? That seems dangerous. Maybe you should make a typedef "LLDBScriptObjectPointer" or whatever, and that's what the SB API returns. Then you could bind to that in the SWIG type map.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143236/new/
https://reviews.llvm.org/D143236
More information about the lldb-commits
mailing list