summaryrefslogtreecommitdiffstats
path: root/lldb/packages/Python/lldbsuite/test/functionalities/inferior-assert
diff options
context:
space:
mode:
authorZachary Turner <zturner@google.com>2016-02-02 22:22:53 +0000
committerZachary Turner <zturner@google.com>2016-02-02 22:22:53 +0000
commitc67466054f8dd467283f17fd31c2c7763ab25016 (patch)
treef318ae510a6696e2d9173e0ad582a1197d64da4c /lldb/packages/Python/lldbsuite/test/functionalities/inferior-assert
parent3a92c35539bd729e12bed95c65c2878f7fb93983 (diff)
downloadbcm5719-llvm-c67466054f8dd467283f17fd31c2c7763ab25016.tar.gz
bcm5719-llvm-c67466054f8dd467283f17fd31c2c7763ab25016.zip
Don't return a tuple from the skip test function.
Previously we were returning a tuple of (bool, skip_reason) from the tuple function. This makes for some awkward code, especially since a value of True for the first argument implies that the second argument is None, and a value of False implies that the second argument is not None. So it was basically redundant, and with this patch we simply return the skip reason or None directly. llvm-svn: 259590
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/functionalities/inferior-assert')
-rw-r--r--lldb/packages/Python/lldbsuite/test/functionalities/inferior-assert/TestInferiorAssert.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/lldb/packages/Python/lldbsuite/test/functionalities/inferior-assert/TestInferiorAssert.py b/lldb/packages/Python/lldbsuite/test/functionalities/inferior-assert/TestInferiorAssert.py
index aba8002ee38..14e09637e38 100644
--- a/lldb/packages/Python/lldbsuite/test/functionalities/inferior-assert/TestInferiorAssert.py
+++ b/lldb/packages/Python/lldbsuite/test/functionalities/inferior-assert/TestInferiorAssert.py
@@ -60,8 +60,8 @@ class AssertingInferiorTestCase(TestBase):
lldbutil.run_break_set_by_file_and_line (self, "main.c", line, num_expected_locations=1, loc_exact=True)
def check_stop_reason(self):
- match_result, _ = matchAndroid(api_levels=list(range(1, 16+1)))(self)
- if match_result:
+ match_failure_reason = matchAndroid(api_levels=list(range(1, 16+1)))(self)
+ if match_failure_reason is None:
# On android until API-16 the abort() call ended in a sigsegv instead of in a sigabrt
stop_reason = 'stop reason = signal SIGSEGV'
else:
OpenPOWER on IntegriCloud