From 349de9331c1b9076fe0aa9b2fd2243fdd1e4255e Mon Sep 17 00:00:00 2001
From: Nikolai Kosjar <nikolai.kosjar@digia.com>
Date: Thu, 13 Mar 2014 10:58:18 -0300
Subject: [PATCH] C++: Fix weird logic in usage of
 TopLevelDeclarationProcessor::processDeclaration

As the name function name suggests, true should be returned if the
declaration should be processed. Otherwise false.

Change-Id: I8d266d99c579b331fee8772bde47aa1a466dae9c
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>
---
 src/libs/3rdparty/cplusplus/Parser.cpp         | 2 +-
 src/plugins/cpptools/cpptoolseditorsupport.cpp | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/libs/3rdparty/cplusplus/Parser.cpp b/src/libs/3rdparty/cplusplus/Parser.cpp
index 2f0bb6e477..bf19628fd2 100644
--- a/src/libs/3rdparty/cplusplus/Parser.cpp
+++ b/src/libs/3rdparty/cplusplus/Parser.cpp
@@ -541,7 +541,7 @@ bool Parser::parseTranslationUnit(TranslationUnitAST *&node)
 
 
         if (TopLevelDeclarationProcessor *processor = _control->topLevelDeclarationProcessor()) {
-            if (processor->processDeclaration(declaration))
+            if (!processor->processDeclaration(declaration))
                 break;
         }
 
diff --git a/src/plugins/cpptools/cpptoolseditorsupport.cpp b/src/plugins/cpptools/cpptoolseditorsupport.cpp
index 451ff8eb8d..664d6761a5 100644
--- a/src/plugins/cpptools/cpptoolseditorsupport.cpp
+++ b/src/plugins/cpptools/cpptoolseditorsupport.cpp
@@ -584,7 +584,7 @@ void CppEditorSupport::recalculateSemanticInfoDetached_helper(QFutureInterface<v
         virtual ~TLDProc() {}
         virtual bool processDeclaration(DeclarationAST *ast) {
             Q_UNUSED(ast);
-            return m_theFuture.isCanceled();
+            return !m_theFuture.isCanceled();
         }
     };
 
-- 
GitLab