[Lldb-commits] [lldb] [lldb-dap] Add feature to remember last non-empty expression. (PR #107485)

via lldb-commits lldb-commits at lists.llvm.org
Thu Sep 12 11:00:08 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 247d3ea843cb20d8d75ec781cd603c8ececf8934...616017152f3f0611462e9863273754036b52f7eb lldb/test/API/tools/lldb-dap/evaluate/TestDAP_evaluate.py
``````````

</details>

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

``````````diff
--- TestDAP_evaluate.py	2024-09-12 17:52:32.000000 +0000
+++ TestDAP_evaluate.py	2024-09-12 17:59:35.638421 +0000
@@ -61,15 +61,15 @@
 
         # Expressions at breakpoint 1, which is in main
         self.assertEvaluate("var1", "20")
         # Empty expression should equate to the previous expression.
         if context == "variable":
-          self.assertEvaluate("", "20")
+            self.assertEvaluate("", "20")
         self.assertEvaluate("var2", "21")
         if context == "variable":
-          self.assertEvaluate("", "21")
-          self.assertEvaluate("", "21")
+            self.assertEvaluate("", "21")
+            self.assertEvaluate("", "21")
         self.assertEvaluate("static_int", "42")
         self.assertEvaluate("non_static_int", "43")
         self.assertEvaluate("struct1.foo", "15")
         self.assertEvaluate("struct2->foo", "16")
 
@@ -198,15 +198,19 @@
         self.continue_to_next_stop()
         self.assertEvaluate("my_bool_vec", "size=2")
 
         # Test memory read, especially with 'empty' repeat commands.
         if context == "repl":
-          self.continue_to_next_stop()
-          self.assertEvaluate("memory read &my_ints",
-                              ".*00 00 00 00 02 00 00 00 04 00 00 00 06 00 00 00.*\n.*08 00 00 00 0a 00 00 00 0c 00 00 00 0e 00 00 00.*\n")
-          self.assertEvaluate("",
-                              ".*10 00 00 00 12 00 00 00 14 00 00 00 16 00 00 00.*\n.*18 00 00 00 1a 00 00 00 1c 00 00 00.*\n")
+            self.continue_to_next_stop()
+            self.assertEvaluate(
+                "memory read &my_ints",
+                ".*00 00 00 00 02 00 00 00 04 00 00 00 06 00 00 00.*\n.*08 00 00 00 0a 00 00 00 0c 00 00 00 0e 00 00 00.*\n",
+            )
+            self.assertEvaluate(
+                "",
+                ".*10 00 00 00 12 00 00 00 14 00 00 00 16 00 00 00.*\n.*18 00 00 00 1a 00 00 00 1c 00 00 00.*\n",
+            )
 
     @skipIfWindows
     def test_generic_evaluate_expressions(self):
         # Tests context-less expression evaluations
         self.run_test_evaluate_expressions(enableAutoVariableSummaries=False)

``````````

</details>


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


More information about the lldb-commits mailing list