From aa5dbe77938cc6db7c4cf9dc84e4e296c4382b82 Mon Sep 17 00:00:00 2001
From: Nikolai Kosjar <nikolai.kosjar@digia.com>
Date: Fri, 16 Jan 2015 15:43:42 +0100
Subject: Clang: Clean up ClangCodeModelPlugin

Change-Id: I4a5a91ff91cb0ec2f8eec647c45c42d8c8faa758
Reviewed-by: Erik Verbruggen <erik.verbruggen@theqtcompany.com>
---
 .../clangcodemodel/clangcodemodelplugin.cpp        | 35 ++++++++++------------
 1 file changed, 15 insertions(+), 20 deletions(-)

(limited to 'src/plugins/clangcodemodel/clangcodemodelplugin.cpp')

diff --git a/src/plugins/clangcodemodel/clangcodemodelplugin.cpp b/src/plugins/clangcodemodel/clangcodemodelplugin.cpp
index 39af043053..da1e19e4c8 100644
--- a/src/plugins/clangcodemodel/clangcodemodelplugin.cpp
+++ b/src/plugins/clangcodemodel/clangcodemodelplugin.cpp
@@ -29,24 +29,17 @@
 ****************************************************************************/
 
 #include "clangcodemodelplugin.h"
+
 #include "clangprojectsettingspropertiespage.h"
 #include "pchmanager.h"
 #include "utils.h"
 
-#include <coreplugin/coreconstants.h>
-#include <coreplugin/icore.h>
-#include <coreplugin/imode.h>
-#include <coreplugin/modemanager.h>
-#include <coreplugin/id.h>
-
 #include <cpptools/cppmodelmanager.h>
 
 #include <projectexplorer/projectpanelfactory.h>
 #include <projectexplorer/projectexplorer.h>
 #include <projectexplorer/session.h>
 
-#include <QtPlugin>
-
 namespace ClangCodeModel {
 namespace Internal {
 
@@ -55,32 +48,34 @@ bool ClangCodeModelPlugin::initialize(const QStringList &arguments, QString *err
     Q_UNUSED(arguments)
     Q_UNUSED(errorMessage)
 
+    // Register widget for project panel
     auto panelFactory = new ProjectExplorer::ProjectPanelFactory();
     panelFactory->setPriority(60);
     panelFactory->setDisplayName(ClangProjectSettingsWidget::tr("Clang Settings"));
     panelFactory->setSimpleCreateWidgetFunction<ClangProjectSettingsWidget>(QIcon());
-
     ProjectExplorer::ProjectPanelFactory::registerFactory(panelFactory);
 
+    // Initialize Clang
     ClangCodeModel::Internal::initializeClang();
 
-    PchManager *pchManager = new PchManager(this);
-
+    // Set up Indexer
+    auto cppModelManager = CppTools::CppModelManager::instance();
 #ifdef CLANG_INDEXING
     m_indexer.reset(new ClangIndexer);
-    CppTools::CppModelManager::instance()->setIndexingSupport(m_indexer->indexingSupport());
+    cppModelManager->setIndexingSupport(m_indexer->indexingSupport());
 #endif // CLANG_INDEXING
 
-    // wire up the pch manager
-    QObject *session = ProjectExplorer::SessionManager::instance();
-    connect(session, SIGNAL(aboutToRemoveProject(ProjectExplorer::Project*)),
-            pchManager, SLOT(onAboutToRemoveProject(ProjectExplorer::Project*)));
-    connect(CppTools::CppModelManager::instance(), SIGNAL(projectPartsUpdated(ProjectExplorer::Project*)),
-            pchManager, SLOT(onProjectPartsUpdated(ProjectExplorer::Project*)));
+    // Set up PchManager
+    PchManager *pchManager = new PchManager(this);
+    ProjectExplorer::SessionManager *sessionManager = ProjectExplorer::SessionManager::instance();
+    connect(sessionManager, &ProjectExplorer::SessionManager::aboutToRemoveProject,
+            pchManager, &PchManager::onAboutToRemoveProject);
+    connect(cppModelManager, &CppTools::CppModelManager::projectPartsUpdated,
+            pchManager, &PchManager::onProjectPartsUpdated);
 
+    // Register ModelManagerSupport
     m_modelManagerSupport.reset(new ModelManagerSupport);
-    CppTools::CppModelManager::instance()->addModelManagerSupport(
-                m_modelManagerSupport.data());
+    cppModelManager->addModelManagerSupport(m_modelManagerSupport.data());
 
     return true;
 }
-- 
cgit v1.2.1