[llvm-branch-commits] [lldb] 4e814b8 - [lldb] Fix typo in SBUnixSignals.get_unix_signals_list (#213527)
Douglas Yung via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Sat Aug 8 22:55:04 PDT 2026
Author: Nerixyz
Date: 2026-08-09T05:54:49Z
New Revision: 4e814b8af6d8c7bb7e29e871613faf4aec065925
URL: https://github.com/llvm/llvm-project/commit/4e814b8af6d8c7bb7e29e871613faf4aec065925
DIFF: https://github.com/llvm/llvm-project/commit/4e814b8af6d8c7bb7e29e871613faf4aec065925.diff
LOG: [lldb] Fix typo in SBUnixSignals.get_unix_signals_list (#213527)
Found when looking through the generated Python file. `sig` doesn't
exist in that context, it should be `idx`.
(cherry picked from commit def770df8325fb60107c24a2de3068aa762754ab)
Added:
Modified:
lldb/bindings/interface/SBUnixSignalsExtensions.i
lldb/test/API/python_api/signals/TestSignalsAPI.py
Removed:
################################################################################
diff --git a/lldb/bindings/interface/SBUnixSignalsExtensions.i b/lldb/bindings/interface/SBUnixSignalsExtensions.i
index 31c7351f1b5dc..ee6ce43e5075e 100644
--- a/lldb/bindings/interface/SBUnixSignalsExtensions.i
+++ b/lldb/bindings/interface/SBUnixSignalsExtensions.i
@@ -11,7 +11,7 @@
def get_unix_signals_list(self):
signals = []
for idx in range(0, self.GetNumSignals()):
- signals.append(self.GetSignalAtIndex(sig))
+ signals.append(self.GetSignalAtIndex(idx))
return signals
threads = property(get_unix_signals_list, None, doc='''A read only property that returns a list() of valid signal numbers for this platform.''')
diff --git a/lldb/test/API/python_api/signals/TestSignalsAPI.py b/lldb/test/API/python_api/signals/TestSignalsAPI.py
index 4b6c7e4214455..f4c967e4b5692 100644
--- a/lldb/test/API/python_api/signals/TestSignalsAPI.py
+++ b/lldb/test/API/python_api/signals/TestSignalsAPI.py
@@ -50,3 +50,9 @@ def test_ignore_signal(self):
self.assertEqual(
process.GetExitStatus(), 0, "The process should have returned 0"
)
+
+ self.assertIn(
+ sigint,
+ unix_signals.get_unix_signals_list(),
+ "SIGINT should be in the list of supported signals",
+ )
More information about the llvm-branch-commits
mailing list