diff options
| author | Zachary Turner <zturner@google.com> | 2016-02-02 22:22:53 +0000 |
|---|---|---|
| committer | Zachary Turner <zturner@google.com> | 2016-02-02 22:22:53 +0000 |
| commit | c67466054f8dd467283f17fd31c2c7763ab25016 (patch) | |
| tree | f318ae510a6696e2d9173e0ad582a1197d64da4c /lldb/packages/Python/lldbsuite/test/functionalities/inferior-assert | |
| parent | 3a92c35539bd729e12bed95c65c2878f7fb93983 (diff) | |
| download | bcm5719-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.py | 4 |
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: |

