From e8510501d85e17f69bd1a32270f527257044ae67 Mon Sep 17 00:00:00 2001 From: Jarek Kobus Date: Wed, 16 Aug 2023 22:43:41 +0200 Subject: [PATCH] Valgrind: Fix includes Change-Id: I30248edc9f536cd3865799dd0b56c1360e72aa2f Reviewed-by: hjk --- .../valgrind/callgrind/callgrindcostitem.cpp | 2 - .../callgrind/callgrindcycledetection.cpp | 4 +- .../valgrind/callgrind/callgrinddatamodel.cpp | 4 -- .../valgrind/callgrind/callgrindfunction.cpp | 6 +-- .../callgrind/callgrindfunctioncycle.cpp | 5 +-- .../valgrind/callgrind/callgrindparsedata.cpp | 1 - .../valgrind/callgrind/callgrindparser.cpp | 5 +-- .../callgrind/callgrindproxymodel.cpp | 2 - .../valgrind/callgrindcostdelegate.cpp | 3 +- src/plugins/valgrind/callgrindcostview.cpp | 10 +---- src/plugins/valgrind/callgrindengine.cpp | 9 +--- src/plugins/valgrind/callgrindtextmark.cpp | 5 +-- src/plugins/valgrind/callgrindtool.cpp | 41 ++++++++----------- .../valgrind/callgrindvisualisation.cpp | 10 ++--- src/plugins/valgrind/memcheckerrorview.cpp | 1 - src/plugins/valgrind/memchecktool.cpp | 34 ++++++--------- src/plugins/valgrind/valgrindengine.cpp | 8 +--- src/plugins/valgrind/valgrindengine.h | 2 +- src/plugins/valgrind/valgrindsettings.cpp | 9 +--- .../valgrind/valgrindtestrunnertest.cpp | 9 +--- .../valgrind/xmlprotocol/errorlistmodel.cpp | 4 +- .../valgrind/xmlprotocol/stackmodel.cpp | 3 -- 22 files changed, 51 insertions(+), 126 deletions(-) diff --git a/src/plugins/valgrind/callgrind/callgrindcostitem.cpp b/src/plugins/valgrind/callgrind/callgrindcostitem.cpp index 40aa56de31e..d6fa4f99013 100644 --- a/src/plugins/valgrind/callgrind/callgrindcostitem.cpp +++ b/src/plugins/valgrind/callgrind/callgrindcostitem.cpp @@ -6,8 +6,6 @@ #include "callgrindparsedata.h" #include "callgrindfunctioncall.h" -#include -#include #include namespace Valgrind::Callgrind { diff --git a/src/plugins/valgrind/callgrind/callgrindcycledetection.cpp b/src/plugins/valgrind/callgrind/callgrindcycledetection.cpp index c32e0ec9b7b..68f6964c344 100644 --- a/src/plugins/valgrind/callgrind/callgrindcycledetection.cpp +++ b/src/plugins/valgrind/callgrind/callgrindcycledetection.cpp @@ -5,13 +5,11 @@ #include "callgrindfunction.h" #include "callgrindfunctioncall.h" -#include "callgrindparsedata.h" #include "callgrindfunctioncycle.h" +#include "callgrindparsedata.h" #include -#include - namespace Valgrind::Callgrind::Internal { CycleDetection::CycleDetection(ParseData *data) diff --git a/src/plugins/valgrind/callgrind/callgrinddatamodel.cpp b/src/plugins/valgrind/callgrind/callgrinddatamodel.cpp index 49d26262471..a623a6a8c98 100644 --- a/src/plugins/valgrind/callgrind/callgrinddatamodel.cpp +++ b/src/plugins/valgrind/callgrind/callgrinddatamodel.cpp @@ -11,11 +11,7 @@ #include #include -#include -#include -#include #include -#include namespace Valgrind { namespace Callgrind { diff --git a/src/plugins/valgrind/callgrind/callgrindfunction.cpp b/src/plugins/valgrind/callgrind/callgrindfunction.cpp index 0ec72d095d8..4015ae7d5b9 100644 --- a/src/plugins/valgrind/callgrind/callgrindfunction.cpp +++ b/src/plugins/valgrind/callgrind/callgrindfunction.cpp @@ -2,20 +2,16 @@ // SPDX-License-Identifier: LicenseRef-Qt-Commercial OR GPL-3.0-only WITH Qt-GPL-exception-1.0 #include "callgrindfunction.h" -#include "callgrindfunction_p.h" #include "callgrindcostitem.h" +#include "callgrindfunction_p.h" #include "callgrindfunctioncall.h" #include "callgrindparsedata.h" #include "../valgrindtr.h" #include -#include -#include -#include #include -#include namespace Valgrind { namespace Callgrind { diff --git a/src/plugins/valgrind/callgrind/callgrindfunctioncycle.cpp b/src/plugins/valgrind/callgrind/callgrindfunctioncycle.cpp index f4d3fdfdedc..64dd7374bcb 100644 --- a/src/plugins/valgrind/callgrind/callgrindfunctioncycle.cpp +++ b/src/plugins/valgrind/callgrind/callgrindfunctioncycle.cpp @@ -2,14 +2,11 @@ // SPDX-License-Identifier: LicenseRef-Qt-Commercial OR GPL-3.0-only WITH Qt-GPL-exception-1.0 #include "callgrindfunctioncycle.h" -#include "callgrindfunction_p.h" +#include "callgrindfunction_p.h" #include "callgrindfunctioncall.h" #include "callgrindparsedata.h" -#include -#include - namespace Valgrind { namespace Callgrind { diff --git a/src/plugins/valgrind/callgrind/callgrindparsedata.cpp b/src/plugins/valgrind/callgrind/callgrindparsedata.cpp index 7b7f2a3e672..62fc8a7cd2f 100644 --- a/src/plugins/valgrind/callgrind/callgrindparsedata.cpp +++ b/src/plugins/valgrind/callgrind/callgrindparsedata.cpp @@ -10,7 +10,6 @@ #include -#include #include #include diff --git a/src/plugins/valgrind/callgrind/callgrindparser.cpp b/src/plugins/valgrind/callgrind/callgrindparser.cpp index 434e7751083..58964172306 100644 --- a/src/plugins/valgrind/callgrind/callgrindparser.cpp +++ b/src/plugins/valgrind/callgrind/callgrindparser.cpp @@ -3,10 +3,10 @@ #include "callgrindparser.h" -#include "callgrindparsedata.h" -#include "callgrindfunctioncall.h" #include "callgrindcostitem.h" #include "callgrindfunction.h" +#include "callgrindfunctioncall.h" +#include "callgrindparsedata.h" #include #include @@ -14,7 +14,6 @@ #include #include -#include #include // #define DEBUG_PARSER diff --git a/src/plugins/valgrind/callgrind/callgrindproxymodel.cpp b/src/plugins/valgrind/callgrind/callgrindproxymodel.cpp index 9d0c8353e6d..3ba426a93d3 100644 --- a/src/plugins/valgrind/callgrind/callgrindproxymodel.cpp +++ b/src/plugins/valgrind/callgrind/callgrindproxymodel.cpp @@ -10,8 +10,6 @@ #include -#include - namespace Valgrind { namespace Callgrind { diff --git a/src/plugins/valgrind/callgrindcostdelegate.cpp b/src/plugins/valgrind/callgrindcostdelegate.cpp index e6afe5df32f..e6ca54756c8 100644 --- a/src/plugins/valgrind/callgrindcostdelegate.cpp +++ b/src/plugins/valgrind/callgrindcostdelegate.cpp @@ -3,9 +3,8 @@ #include "callgrindcostdelegate.h" -#include "callgrindhelper.h" - #include "callgrind/callgrindabstractmodel.h" +#include "callgrindhelper.h" #include diff --git a/src/plugins/valgrind/callgrindcostview.cpp b/src/plugins/valgrind/callgrindcostview.cpp index 3d54a99c295..4181abf6a99 100644 --- a/src/plugins/valgrind/callgrindcostview.cpp +++ b/src/plugins/valgrind/callgrindcostview.cpp @@ -3,17 +3,11 @@ #include "callgrindcostview.h" +#include "callgrind/callgrinddatamodel.h" +#include "callgrind/callgrindcallmodel.h" #include "callgrindnamedelegate.h" -#include -#include -#include -#include - #include -#include -#include -#include using namespace Valgrind::Callgrind; diff --git a/src/plugins/valgrind/callgrindengine.cpp b/src/plugins/valgrind/callgrindengine.cpp index 192174c6e54..3d86b192db6 100644 --- a/src/plugins/valgrind/callgrindengine.cpp +++ b/src/plugins/valgrind/callgrindengine.cpp @@ -3,11 +3,8 @@ #include "callgrindengine.h" -#include "valgrindsettings.h" - -#include -#include -#include +#include "callgrind/callgrindparser.h" +#include "valgrindtr.h" #include @@ -17,8 +14,6 @@ #include #include -#include - #define CALLGRIND_CONTROL_DEBUG 0 using namespace ProjectExplorer; diff --git a/src/plugins/valgrind/callgrindtextmark.cpp b/src/plugins/valgrind/callgrindtextmark.cpp index 07b6af16c24..39cbf3803a5 100644 --- a/src/plugins/valgrind/callgrindtextmark.cpp +++ b/src/plugins/valgrind/callgrindtextmark.cpp @@ -3,18 +3,15 @@ #include "callgrindtextmark.h" -#include "callgrindhelper.h" - #include "callgrind/callgrinddatamodel.h" #include "callgrind/callgrindfunction.h" +#include "callgrindhelper.h" #include "valgrindtr.h" #include -#include #include #include -#include using namespace Utils; using namespace Valgrind::Internal; diff --git a/src/plugins/valgrind/callgrindtool.cpp b/src/plugins/valgrind/callgrindtool.cpp index fb16a4441ef..219fe800bc1 100644 --- a/src/plugins/valgrind/callgrindtool.cpp +++ b/src/plugins/valgrind/callgrindtool.cpp @@ -3,30 +3,22 @@ #include "callgrindtool.h" +#include "callgrind/callgrindcallmodel.h" +#include "callgrind/callgrinddatamodel.h" +#include "callgrind/callgrindfunction.h" +#include "callgrind/callgrindfunctioncall.h" +#include "callgrind/callgrindparsedata.h" +#include "callgrind/callgrindparser.h" +#include "callgrind/callgrindproxymodel.h" +#include "callgrind/callgrindstackbrowser.h" #include "callgrindcostdelegate.h" #include "callgrindcostview.h" #include "callgrindengine.h" #include "callgrindtextmark.h" #include "callgrindvisualisation.h" +#include "valgrindsettings.h" #include "valgrindtr.h" -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include - -#include -#include -#include -#include -#include - #include #include #include @@ -39,10 +31,10 @@ #include -#include - -#include -#include +#include +#include +#include +#include #include #include @@ -51,23 +43,22 @@ #include #include -#include +#include +#include + #include #include -#include #include #include #include #include #include -#include #include #include #include #include #include -#include #include using namespace Debugger; diff --git a/src/plugins/valgrind/callgrindvisualisation.cpp b/src/plugins/valgrind/callgrindvisualisation.cpp index d0673bbf849..93d129ae48c 100644 --- a/src/plugins/valgrind/callgrindvisualisation.cpp +++ b/src/plugins/valgrind/callgrindvisualisation.cpp @@ -3,22 +3,18 @@ #include "callgrindvisualisation.h" +#include "callgrind/callgrinddatamodel.h" +#include "callgrind/callgrindfunction.h" +#include "callgrind/callgrindproxymodel.h" #include "callgrindhelper.h" #include "valgrindtr.h" -#include -#include -#include -#include - #include #include -#include #include #include #include -#include #include #include #include diff --git a/src/plugins/valgrind/memcheckerrorview.cpp b/src/plugins/valgrind/memcheckerrorview.cpp index ef6ab2856e1..695a1446faa 100644 --- a/src/plugins/valgrind/memcheckerrorview.cpp +++ b/src/plugins/valgrind/memcheckerrorview.cpp @@ -5,7 +5,6 @@ #include "valgrindsettings.h" #include "valgrindtr.h" - #include "xmlprotocol/error.h" #include "xmlprotocol/errorlistmodel.h" #include "xmlprotocol/frame.h" diff --git a/src/plugins/valgrind/memchecktool.cpp b/src/plugins/valgrind/memchecktool.cpp index 599dec740b7..5f5a82df832 100644 --- a/src/plugins/valgrind/memchecktool.cpp +++ b/src/plugins/valgrind/memchecktool.cpp @@ -16,6 +16,14 @@ #include "xmlprotocol/parser.h" #include "xmlprotocol/stack.h" +#include +#include +#include +#include +#include +#include +#include + #include #include #include @@ -35,19 +43,7 @@ #include #include -#include -#include - -#include -#include -#include -#include -#include -#include -#include - #include -#include #include #include #include @@ -55,22 +51,20 @@ #include #include +#include +#include #include -#include #include #include #include #include -#include -#include -#include - -#include -#include #include +#include #include #include #include +#include +#include #include #ifdef Q_OS_WIN @@ -1161,7 +1155,6 @@ MemcheckToolRunner::MemcheckToolRunner(RunControl *runControl) this, &MemcheckToolRunner::startDebugger); connect(&m_runner, &ValgrindRunner::logMessageReceived, this, &MemcheckToolRunner::appendLog); -// m_runner.disableXml(); } else { connect(&m_runner, &ValgrindRunner::internalError, this, &MemcheckToolRunner::internalParserError); @@ -1177,7 +1170,6 @@ MemcheckToolRunner::MemcheckToolRunner(RunControl *runControl) dd->setupRunner(this); } - const char heobProfileC[] = "Heob/Profile"; const char heobProfileNameC[] = "Name"; const char heobXmlC[] = "Xml"; diff --git a/src/plugins/valgrind/valgrindengine.cpp b/src/plugins/valgrind/valgrindengine.cpp index 0b6abb771f9..4af25d2c7ca 100644 --- a/src/plugins/valgrind/valgrindengine.cpp +++ b/src/plugins/valgrind/valgrindengine.cpp @@ -6,23 +6,19 @@ #include "valgrindsettings.h" #include "valgrindtr.h" -#include - #include #include -#include #include +#include + #include #include #include #include -#include -#include #include -using namespace Debugger; using namespace Core; using namespace Utils; using namespace ProjectExplorer; diff --git a/src/plugins/valgrind/valgrindengine.h b/src/plugins/valgrind/valgrindengine.h index d8c5a0ae690..5644e92d9b7 100644 --- a/src/plugins/valgrind/valgrindengine.h +++ b/src/plugins/valgrind/valgrindengine.h @@ -3,10 +3,10 @@ #pragma once +#include "valgrindrunner.h" #include "valgrindsettings.h" #include -#include #include diff --git a/src/plugins/valgrind/valgrindsettings.cpp b/src/plugins/valgrind/valgrindsettings.cpp index 3f81f05b20d..6c23677face 100644 --- a/src/plugins/valgrind/valgrindsettings.cpp +++ b/src/plugins/valgrind/valgrindsettings.cpp @@ -5,26 +5,21 @@ #include "callgrindcostdelegate.h" #include "valgrindtr.h" - -#include +#include "xmlprotocol/error.h" #include + #include #include #include #include #include -#include #include -#include - -#include #include #include #include -#include #include using namespace Utils; diff --git a/src/plugins/valgrind/valgrindtestrunnertest.cpp b/src/plugins/valgrind/valgrindtestrunnertest.cpp index e8678329ed8..5ae6ee04179 100644 --- a/src/plugins/valgrind/valgrindtestrunnertest.cpp +++ b/src/plugins/valgrind/valgrindtestrunnertest.cpp @@ -3,24 +3,19 @@ #include "valgrindtestrunnertest.h" +#include "valgrindrunner.h" #include "xmlprotocol/frame.h" #include "xmlprotocol/stack.h" -#include "valgrindrunner.h" - -#include -#include #include #include #include -#include #include -#include +#include #define HEADER_LENGTH 3 -using namespace ProjectExplorer; using namespace Valgrind::XmlProtocol; using namespace Utils; diff --git a/src/plugins/valgrind/xmlprotocol/errorlistmodel.cpp b/src/plugins/valgrind/xmlprotocol/errorlistmodel.cpp index 96c1819b284..34276dc54c1 100644 --- a/src/plugins/valgrind/xmlprotocol/errorlistmodel.cpp +++ b/src/plugins/valgrind/xmlprotocol/errorlistmodel.cpp @@ -8,14 +8,12 @@ #include "../valgrindtr.h" #include + #include -#include #include #include -#include - namespace Valgrind { namespace XmlProtocol { diff --git a/src/plugins/valgrind/xmlprotocol/stackmodel.cpp b/src/plugins/valgrind/xmlprotocol/stackmodel.cpp index 535ceb38f9b..1027456b8c7 100644 --- a/src/plugins/valgrind/xmlprotocol/stackmodel.cpp +++ b/src/plugins/valgrind/xmlprotocol/stackmodel.cpp @@ -9,9 +9,6 @@ #include -#include -#include - namespace Valgrind { namespace XmlProtocol {