From cb473a48b7afda9f00606d55d57c6986675b8eb2 Mon Sep 17 00:00:00 2001 From: Daniel Marjamaki Date: Wed, 6 Apr 2011 19:49:04 +0200 Subject: [PATCH] Preprocessor: Use Settings::debugwarnings instead of NDEBUG to determine if 'missing system include' should be reported or not --- lib/preprocessor.cpp | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/lib/preprocessor.cpp b/lib/preprocessor.cpp index 227733e95..bfc68a027 100644 --- a/lib/preprocessor.cpp +++ b/lib/preprocessor.cpp @@ -1910,13 +1910,7 @@ void Preprocessor::handleIncludes(std::string &code, const std::string &filePath } else if (!fileOpened) { - // TODO: Fix the handling of system includes and then - // remove the "headerType == UserHeader" -#ifdef NDEBUG - if (headerType == UserHeader && _errorLogger && _settings && _settings->isEnabled("missingInclude")) -#else - if (_errorLogger && _settings && _settings->isEnabled("missingInclude")) -#endif + if (_errorLogger && _settings && ((headerType == UserHeader) && _settings->isEnabled("missingInclude") || _settings->debugwarnings)) { std::string f = filePath;