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

Charles Zablit via lldb-commits lldb-commits at lists.llvm.org
Fri Nov 28 03:36:52 PST 2025


https://github.com/charles-zablit created https://github.com/llvm/llvm-project/pull/169903

`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.

>From 51fe94071cbe1d11af5f396b8ce9accc417bd96c Mon Sep 17 00:00:00 2001
From: Charles Zablit <c_zablit at apple.com>
Date: Fri, 28 Nov 2025 11:33:23 +0000
Subject: [PATCH] [lldb][dotest] use unused variable

---
 lldb/packages/Python/lldbsuite/test/dotest.py | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

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):



More information about the lldb-commits mailing list