[Lldb-commits] [lldb] [lldb] Consolidate ScriptedThreadPlan state into ScriptedMetadata (PR #199064)

[Alex Langford via lldb-commits]

================
@@ -92,8 +97,8 @@ void ScriptedThreadPlan::DidPush() {
 
 bool ScriptedThreadPlan::ShouldStop(Event *event_ptr) {
   Log *log = GetLog(LLDBLog::Thread);
-  LLDB_LOGF(log, "%s called on Scripted Thread Plan: %s )",
-            LLVM_PRETTY_FUNCTION, m_class_name.c_str());
+  LLDB_LOG(log, "{0} called on Scripted Thread Plan: {1} )",
+           LLVM_PRETTY_FUNCTION, GetScriptClassName());
----------------
bulbazord wrote:

Since you're already here, doesn't LLDB_LOG already handle function names in some capacity? Why do we need `LLVM_PRETTY_FUNCTION` here at all?

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