[Lldb-commits] [lldb] [lldb][test] Fix call signature in TestAppleSimulatorOSType (PR #191185)

via lldb-commits lldb-commits at lists.llvm.org
Thu Apr 9 06:06:05 PDT 2026


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Raphael Isemann (Teemperor)

<details>
<summary>Changes</summary>

Commit 918e446ef28ac97df20d6ef2bd50c78e2fe903ac removed the stderr_lines_to_read argument but didn't adjust the call site in TestAppleSimulatorOSType. This patch just removes the extra arg here too.

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


1 Files Affected:

- (modified) lldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py (-4) 


``````````diff
diff --git a/lldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py b/lldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py
index 3dc11a3ff74f2..9e59ffd5db6e2 100644
--- a/lldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py
+++ b/lldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py
@@ -13,9 +13,6 @@
 class TestAppleSimulatorOSType(gdbremote_testcase.GdbRemoteTestCaseBase):
     SHARED_BUILD_TESTCASE = False
 
-    # Number of stderr lines to read from the simctl output.
-    READ_LINES = 10
-
     def check_simulator_ostype(self, sdk, platform_name, arch=platform.machine()):
         # Get simulator
         deviceUDID = None
@@ -59,7 +56,6 @@ def check_simulator_ostype(self, sdk, platform_name, arch=platform.machine()):
             deviceUDID,
             self.getBuildArtifact(exe_name),
             ["print-pid", "sleep:10"],
-            self.READ_LINES,
             [r"PID: (.*)"],
             self.trace,
         )

``````````

</details>


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


More information about the lldb-commits mailing list