[Lldb-commits] [lldb] [lldb-dap] Improve `stackTrace` and `exceptionInfo` DAP request handlers (PR #105905)

via lldb-commits lldb-commits at lists.llvm.org
Wed Aug 28 14:31:11 PDT 2024


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 cdd11d694a406a98a16d6265168ee2fbe1b6a87c...7960402faaeabeba610cc56cba04638e355563c8 lldb/test/API/tools/lldb-dap/exception/cpp/TestDAP_exception_cpp.py lldb/test/API/tools/lldb-dap/exception/objc/TestDAP_exception_objc.py lldb/test/API/tools/lldb-dap/extendedStackTrace/TestDAP_extendedStackTrace.py lldb/packages/Python/lldbsuite/test/lldbplatformutil.py lldb/packages/Python/lldbsuite/test/tools/lldb-dap/dap_server.py lldb/packages/Python/lldbsuite/test/tools/lldb-dap/lldbdap_testcase.py lldb/test/API/tools/lldb-dap/exception/TestDAP_exception.py lldb/test/API/tools/lldb-dap/stackTrace/TestDAP_stackTrace.py lldb/test/API/tools/lldb-dap/stackTraceMissingFunctionName/TestDAP_stackTraceMissingFunctionName.py
``````````

</details>

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

``````````diff
--- test/API/tools/lldb-dap/stackTrace/TestDAP_stackTrace.py	2024-08-28 21:25:57.000000 +0000
+++ test/API/tools/lldb-dap/stackTrace/TestDAP_stackTrace.py	2024-08-28 21:30:41.219672 +0000
@@ -85,19 +85,25 @@
         frameCount = len(stackFrames)
         self.assertGreaterEqual(
             frameCount, 40, "verify we get at least 40 frames for all frames"
         )
         self.assertEqual(
-            totalFrames, frameCount, "verify total frames returns a speculative page size"
+            totalFrames,
+            frameCount,
+            "verify total frames returns a speculative page size",
         )
         self.verify_stackFrames(startFrame, stackFrames)
 
         # Verify totalFrames contains a speculative page size of additional frames with startFrame = 0 and levels = 0
-        (stackFrames, totalFrames) = self.get_stackFrames_and_totalFramesCount(startFrame=0, levels=10)
+        (stackFrames, totalFrames) = self.get_stackFrames_and_totalFramesCount(
+            startFrame=0, levels=10
+        )
         self.assertEqual(len(stackFrames), 10, "verify we get levels=10 frames")
         self.assertEqual(
-            totalFrames, len(stackFrames) + self.page_size, "verify total frames returns a speculative page size"
+            totalFrames,
+            len(stackFrames) + self.page_size,
+            "verify total frames returns a speculative page size",
         )
         self.verify_stackFrames(startFrame, stackFrames)
 
         # Verify all stack frames by specifying startFrame = 0 and levels not
         # specified

``````````

</details>


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


More information about the lldb-commits mailing list