[Lldb-commits] [lldb] [lldb][dotest] use unused variable (PR #169903)

[via lldb-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Charles Zablit (charles-zablit)

<details>
<summary>Changes</summary>

`skipped_categories_list` was unused in `dotest.py`. This patch ensures it's used where intended.

2238dcc39358353cac21df75c3c3286ab20b8f53 switched to using `configuration.skip_categories` directly when reformatting the code, making `skipped_categories_list` unused.

---
Full diff: https://github.com/llvm/llvm-project/pull/169903.diff


1 Files Affected:

- (modified) lldb/packages/Python/lldbsuite/test/dotest.py (+1-5) 


``````````diff
diff --git a/lldb/packages/Python/lldbsuite/test/dotest.py b/lldb/packages/Python/lldbsuite/test/dotest.py
index 63f7df4de1894..f280bd2b3887b 100644
--- a/lldb/packages/Python/lldbsuite/test/dotest.py
+++ b/lldb/packages/Python/lldbsuite/test/dotest.py
@@ -1108,11 +1108,7 @@ def run_suite():
     checkDAPSupport()
 
     skipped_categories_list = ", ".join(configuration.skip_categories)
-    print(
-        "Skipping the following test categories: {}".format(
-            configuration.skip_categories
-        )
-    )
+    print(f"Skipping the following test categories: {skipped_categories_list}")
 
     for testdir in configuration.testdirs:
         for dirpath, dirnames, filenames in os.walk(testdir):

``````````

</details>


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