Fix lints

Author: qxy11

lldb/packages/Python/lldbsuite/test/builders/builder.py

@@ -247,7 +247,7 @@ def getLLDBObjRoot(self):
     def _getDebugInfoArgs(self, debug_info):
         if debug_info is None:
             return []
-        
+
         debug_options = debug_info if isinstance(debug_info, list) else [debug_info]
         option_flags = {
             "dwarf": {"MAKE_DSYM": "NO"},
@@ -256,15 +256,15 @@ def _getDebugInfoArgs(self, debug_info):
             "debug_names": {"MAKE_DEBUG_NAMES": "YES"},
             "dwp": {"MAKE_DSYM": "NO", "MAKE_DWP": "YES"},
         }
-        
+
         # Collect all flags, with later options overriding earlier ones
         flags = {}
-        
+
         for option in debug_options:
             if not option or option not in option_flags:
-                return None # Invalid options
+                return None  # Invalid options
             flags.update(option_flags[option])
-        
+
         return [f"{key}={value}" for key, value in flags.items()]
 
     def getBuildCommand(

lldb/test/API/commands/statistics/basic/TestStats.py

@@ -540,11 +540,11 @@ def test_non_split_dwarf_has_no_dwo_files(self):
         debug_stats = self.get_stats()
         self.assertIn("totalDwoFileCount", debug_stats)
         self.assertIn("totalLoadedDwoFileCount", debug_stats)
-        
+
         # Verify that the dwo file count is zero
         self.assertEqual(debug_stats["totalDwoFileCount"], 0)
         self.assertEqual(debug_stats["totalLoadedDwoFileCount"], 0)
-    
+
     @add_test_categories(["dwo"])
     def test_no_debug_names_eager_loads_dwo_files(self):
         """
@@ -554,7 +554,7 @@ def test_no_debug_names_eager_loads_dwo_files(self):
         - With debug_names: DebugNamesDWARFIndex allows for lazy loading.
           DWO files are loaded on-demand when symbols are actually looked up
         - Without debug_names: ManualDWARFIndex uses eager loading.
-          All DWO files are loaded upfront during the first symbol lookup to build a manual index. 
+          All DWO files are loaded upfront during the first symbol lookup to build a manual index.
         """
         da = {"CXX_SOURCES": "third.cpp baz.cpp", "EXE": self.getBuildArtifact("a.out")}
         self.build(dictionary=da, debug_info=["dwo"])
@@ -564,7 +564,7 @@ def test_no_debug_names_eager_loads_dwo_files(self):
         debug_stats = self.get_stats()
         self.assertIn("totalDwoFileCount", debug_stats)
         self.assertIn("totalLoadedDwoFileCount", debug_stats)
-        
+
         # Verify that all DWO files are loaded
         self.assertEqual(debug_stats["totalDwoFileCount"], 2)
         self.assertEqual(debug_stats["totalLoadedDwoFileCount"], 2)
@@ -594,13 +594,13 @@ def test_split_dwarf_dwo_file_count(self):
         self.assertIn("totalDwoFileCount", debug_stats)
         self.assertEqual(debug_stats["totalLoadedDwoFileCount"], 0)
         self.assertEqual(debug_stats["totalDwoFileCount"], 2)
-        
+
         # Since there's only one module, module stats should have the same counts as total counts
         self.assertIn("dwoFileCount", debug_stats["modules"][0])
         self.assertIn("loadedDwoFileCount", debug_stats["modules"][0])
         self.assertEqual(debug_stats["modules"][0]["loadedDwoFileCount"], 0)
         self.assertEqual(debug_stats["modules"][0]["dwoFileCount"], 2)
-        
+
         # 2) Setting breakpoint in main triggers loading of third.dwo (contains main function)
         self.runCmd("b main")
         debug_stats = self.get_stats()
@@ -609,7 +609,7 @@ def test_split_dwarf_dwo_file_count(self):
 
         self.assertEqual(debug_stats["modules"][0]["loadedDwoFileCount"], 1)
         self.assertEqual(debug_stats["modules"][0]["dwoFileCount"], 2)
-        
+
         # 3) Type lookup forces loading of baz.dwo (contains struct Baz definition)
         self.runCmd("type lookup Baz")
         debug_stats = self.get_stats()
@@ -618,7 +618,7 @@ def test_split_dwarf_dwo_file_count(self):
 
         self.assertEqual(debug_stats["modules"][0]["loadedDwoFileCount"], 2)
         self.assertEqual(debug_stats["modules"][0]["dwoFileCount"], 2)
-    
+
     @add_test_categories(["dwo"])
     def test_dwp_dwo_file_count(self):
         """
@@ -627,7 +627,7 @@ def test_dwp_dwo_file_count(self):
         verifies the loaded dwo file count is the expected count after running
         various commands.
 
-        We expect the DWO file counters to reflect the number of compile units 
+        We expect the DWO file counters to reflect the number of compile units
         loaded from the DWP file (each representing what was originally a separate DWO file)
         """
         da = {"CXX_SOURCES": "third.cpp baz.cpp", "EXE": self.getBuildArtifact("a.out")}
@@ -642,19 +642,18 @@ def test_dwp_dwo_file_count(self):
         self.assertIn("totalDwoFileCount", debug_stats)
         self.assertEqual(debug_stats["totalLoadedDwoFileCount"], 0)
         self.assertEqual(debug_stats["totalDwoFileCount"], 2)
-        
+
         # Setting breakpoint in main triggers parsing of the CU within a.dwp corresponding to third.dwo (contains main function)
         self.runCmd("b main")
         debug_stats = self.get_stats()
         self.assertEqual(debug_stats["totalLoadedDwoFileCount"], 1)
         self.assertEqual(debug_stats["totalDwoFileCount"], 2)
-        
+
         # Type lookup forces parsing of the CU within a.dwp corresponding to baz.dwo (contains struct Baz definition)
         self.runCmd("type lookup Baz")
         debug_stats = self.get_stats()
         self.assertEqual(debug_stats["totalDwoFileCount"], 2)
         self.assertEqual(debug_stats["totalLoadedDwoFileCount"], 2)
-        
 
     @skipUnlessDarwin
     @no_debug_info_test