summaryrefslogtreecommitdiffstats
path: root/Source/CTest/cmCTestRunTest.cxx
diff options
context:
space:
mode:
authorBrad King <brad.king@kitware.com>2019-08-22 15:32:54 (GMT)
committerBrad King <brad.king@kitware.com>2019-08-26 13:43:46 (GMT)
commit303e81343804d5ef271a3669c1e6cac558055ca1 (patch)
tree84f1be553d209509b1f095619d2e421199c83baa /Source/CTest/cmCTestRunTest.cxx
parent51565abe7929f4baf5772345dc8d9c62ece9cf7a (diff)
downloadCMake-303e81343804d5ef271a3669c1e6cac558055ca1.zip
CMake-303e81343804d5ef271a3669c1e6cac558055ca1.tar.gz
CMake-303e81343804d5ef271a3669c1e6cac558055ca1.tar.bz2
CTest: Simplify some boolean conditions
Directly compare two boolean values instead of spelling out accepted combinations. Issue: #19610
Diffstat (limited to 'Source/CTest/cmCTestRunTest.cxx')
-rw-r--r--Source/CTest/cmCTestRunTest.cxx3
1 files changed, 1 insertions, 2 deletions
diff --git a/Source/CTest/cmCTestRunTest.cxx b/Source/CTest/cmCTestRunTest.cxx
index fb91322..d232c23 100644
--- a/Source/CTest/cmCTestRunTest.cxx
+++ b/Source/CTest/cmCTestRunTest.cxx
@@ -148,8 +148,7 @@ bool cmCTestRunTest::EndTest(size_t completed, size_t total, bool started)
this->TestResult.CompletionStatus = s.str();
cmCTestLog(this->CTest, HANDLER_OUTPUT, "***Skipped ");
skipped = true;
- } else if ((success && !this->TestProperties->WillFail) ||
- (!success && this->TestProperties->WillFail)) {
+ } else if (success != this->TestProperties->WillFail) {
this->TestResult.Status = cmCTestTestHandler::COMPLETED;
outputStream << " Passed ";
} else {