[Lldb-commits] [lldb] Default transcript dumping in "statistics dump" to false (PR #145436)

Michael Buch via lldb-commits lldb-commits at lists.llvm.org
Mon Jun 23 23:54:37 PDT 2025


================
@@ -193,9 +193,8 @@ struct StatisticsOptions {
   bool GetIncludeTranscript() const {
     if (m_include_transcript.has_value())
       return m_include_transcript.value();
-    // `m_include_transcript` has no value set, so return a value based on
-    // `m_summary_only`.
-    return !GetSummaryOnly();
+    // Default to false in both default mode and summary mode.
+    return false;
----------------
Michael137 wrote:

Can we just do:
```
return m_include_transcript.value_or(false);
```

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


More information about the lldb-commits mailing list