[Lldb-commits] [PATCH] D148548: [lldb] Improve breakpoint management for interactive scripted process

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Apr 20 17:22:29 PDT 2023


JDevlieghere added a comment.

LGTM but I'll hold off on accepting to give Alex a change to take another look as he had more significant comments.



================
Comment at: lldb/source/Interpreter/ScriptInterpreter.cpp:78-91
 lldb::DataExtractorSP
 ScriptInterpreter::GetDataExtractorFromSBData(const lldb::SBData &data) const {
   return data.m_opaque_sp;
 }
 
+lldb::BreakpointSP ScriptInterpreter::GetOpaqueTypeFromSBBreakpoint(
+    const lldb::SBBreakpoint &breakpoint) const {
----------------
NTBF for this patch but we should really be using templates here to avoid code duplication. 


================
Comment at: lldb/source/Plugins/ScriptInterpreter/Python/ScriptedProcessPythonInterface.cpp:124
+  if (!CheckStructuredDataObject(LLVM_PRETTY_FUNCTION, obj, error))
+    return {};
+
----------------
return false


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148548/new/

https://reviews.llvm.org/D148548



More information about the lldb-commits mailing list