[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