Only check DiagnosticsEngine::hasErrorOccurred, hasFatalErrorOccurred…

… cannot be true without it.
root-project · Feb 17, 2017 · 097a4e3 · 097a4e3
1 parent de62978
commit 097a4e3
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/lib/Interpreter/IncrementalParser.cpp b/lib/Interpreter/IncrementalParser.cpp
@@ -371,8 +371,11 @@ namespace cling {
     //TODO: Make the enum orable.
     EParseResult ParseResult = kSuccess;
 
-    if (Diags.hasErrorOccurred() || Diags.hasFatalErrorOccurred()
-        || T->getIssuedDiags() == Transaction::kErrors) {
+    assert((Diags.hasFatalErrorOccurred() ? Diags.hasErrorOccurred() : true)
+            && "Diags.hasFatalErrorOccurred without Diags.hasErrorOccurred !");
+
+    if (Diags.hasErrorOccurred() || T->getIssuedDiags() == Transaction::kErrors)
+    {
       T->setIssuedDiags(Transaction::kErrors);
       ParseResult = kFailed;
     } else if (Diags.getNumWarnings() > 0) {