[Lldb-commits] [lldb] [lldb] Avoid force loading symbols in statistics collection (PR #136236)

[via lldb-commits]

github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {darker}-->


:warning: Python code formatter, darker found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
darker --check --diff -r HEAD~1...HEAD lldb/test/API/commands/statistics/basic/TestStats.py
``````````

</details>

<details>
<summary>
View the diff from darker here.
</summary>

``````````diff
--- TestStats.py	2025-04-21 22:51:10.000000 +0000
+++ TestStats.py	2025-04-21 22:53:08.000273 +0000
@@ -207,11 +207,10 @@
             self.verify_keys(
                 module_stats, '"module_stats"', module_stat_keys_exist, None
             )
             self.assertGreater(module_stats["symbolsLoaded"], 0)
 
-
     def test_default_no_run_no_preload_symbols(self):
         """Test "statistics dump" without running the target and without
         preloading symbols.
 
         Checks that symbol count are zero.
@@ -230,11 +229,10 @@
         self.assertEqual(debug_stats["totalSymbolsLoaded"], 0)
 
         # No symbols should be loaded in each module
         for module_stats in debug_stats["modules"]:
             self.assertEqual(module_stats["symbolsLoaded"], 0)
-
 
     def test_default_with_run(self):
         """Test "statistics dump" when running the target to a breakpoint.
 
         When we run the target, we expect to see 'launchOrAttachTime' and

``````````

</details>


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