Fix: Regression in the `cmConditionEvaluator::HandleLevel0`

As reported in the BUG #22524, mismatched parenthesis reported differently
for `while()` and `if()`.

The problem was in the double loop (over "handlers" and the arguments),
where the outer loop didn't check the result of the running handler.

1 files changed, 7 insertions, 1 deletions

diff --git a/Source/cmConditionEvaluator.cxx b/Source/cmConditionEvaluator.cxx
index 49189d8..68bc4d8 100644
--- a/Source/cmConditionEvaluator.cxx
+++ b/Source/cmConditionEvaluator.cxx
@@ -260,11 +260,17 @@ bool cmConditionEvaluator::IsTrue(
   for (auto fn : handlers) {
     // Call the reducer 'till there is anything to reduce...
     // (i.e., if after an iteration the size becomes smaller)
+    auto levelResult = true;
     for (auto beginSize = newArgs.size();
-         (this->*fn)(newArgs, errorString, status) &&
+         (levelResult = (this->*fn)(newArgs, errorString, status)) &&
          newArgs.size() < beginSize;
          beginSize = newArgs.size()) {
     }
+
+    if (!levelResult) {
+      // NOTE `errorString` supposed to be set already
+      return false;
+    }
   }
 
   // now at the end there should only be one argument left