[Lldb-commits] [lldb] [LLDB] Fix type formatting empty c-strings (PR #68924)

via lldb-commits lldb-commits at lists.llvm.org
Thu Oct 12 13:11:50 PDT 2023


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 220244b71ba2a0301bb13fb195d64a66418d1c70..eaf314f813202fd71b884f3cd2e87db6cfd97e96 lldb/test/API/functionalities/data-formatter/builtin-formats/TestBuiltinFormats.py
``````````

</details>

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

``````````diff
--- TestBuiltinFormats.py	2023-10-12 20:00:08.000000 +0000
+++ TestBuiltinFormats.py	2023-10-12 20:11:42.887284 +0000
@@ -25,13 +25,12 @@
         self.build()
         lldbutil.run_to_source_breakpoint(
             self, "// break here", lldb.SBFileSpec("main.cpp")
         )
         # We can dump correctly non char* c-strings with explicit formatting.
-        self.assertIn(' = ""', self.getFormatted("c-string", "void_empty_cstring"));
-        self.assertIn(' = ""', self.getFormatted("c-string", "empty_cstring"));
-
+        self.assertIn(' = ""', self.getFormatted("c-string", "void_empty_cstring"))
+        self.assertIn(' = ""', self.getFormatted("c-string", "empty_cstring"))
 
     # TODO: Move as many asserts as possible within this function to `testAllPlatforms`.
     # Currently `arm` is being skipped even though many asserts would effectively
     # pass.
     @no_debug_info_test

``````````

</details>


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


More information about the lldb-commits mailing list