[Lldb-commits] [lldb] [lldb] Add support for ScriptedFrame to provide values/variables. (PR #178575)

[Aman LaChapelle via lldb-commits]

================
@@ -265,3 +269,63 @@ lldb::RegisterContextSP ScriptedFrame::GetRegisterContext() {
 
   return m_reg_context_sp;
 }
+
+VariableList *ScriptedFrame::GetVariableList(bool get_file_globals,
+                                             Status *error_ptr) {
+  // Call into GetInScopeVariableList to populate m_variable_list_sp.
+  m_variable_list_sp = GetInScopeVariableList(get_file_globals, false);
----------------
bzcheeseman wrote:

I didn't want to do that either because I'd be creating a shared ptr from a raw pointer so it risks a double-free. I'm going to put together a function to populate m_variable_list_sp and then call *that* from both places.

https://github.com/llvm/llvm-project/pull/178575