Move CanonicalSymbol from CppEditor to CppTools

Needed for a follow-up change.

Change-Id: Ibb4815f3411f0d63deac8c32583178470668f67b
Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
This commit is contained in:
Nikolai Kosjar
2017-05-23 15:38:16 +02:00
parent d84a6ed411
commit 6174cca5a0
8 changed files with 14 additions and 16 deletions

View File

@@ -26,7 +26,6 @@
#include "cppeditor.h" #include "cppeditor.h"
#include "cppautocompleter.h" #include "cppautocompleter.h"
#include "cppcanonicalsymbol.h"
#include "cppdocumentationcommenthelper.h" #include "cppdocumentationcommenthelper.h"
#include "cppeditorconstants.h" #include "cppeditorconstants.h"
#include "cppeditordocument.h" #include "cppeditordocument.h"
@@ -47,6 +46,7 @@
#include <coreplugin/editormanager/documentmodel.h> #include <coreplugin/editormanager/documentmodel.h>
#include <coreplugin/infobar.h> #include <coreplugin/infobar.h>
#include <cpptools/cppcanonicalsymbol.h>
#include <cpptools/cppchecksymbols.h> #include <cpptools/cppchecksymbols.h>
#include <cpptools/cppcodeformatter.h> #include <cpptools/cppcodeformatter.h>
#include <cpptools/cppcompletionassistprovider.h> #include <cpptools/cppcompletionassistprovider.h>

View File

@@ -3,7 +3,6 @@ include(../../qtcreatorplugin.pri)
HEADERS += \ HEADERS += \
cppautocompleter.h \ cppautocompleter.h \
cppcanonicalsymbol.h \
cppcodemodelinspectordialog.h \ cppcodemodelinspectordialog.h \
cppdocumentationcommenthelper.h \ cppdocumentationcommenthelper.h \
cppeditor.h \ cppeditor.h \
@@ -35,7 +34,6 @@ HEADERS += \
SOURCES += \ SOURCES += \
cppautocompleter.cpp \ cppautocompleter.cpp \
cppcanonicalsymbol.cpp \
cppcodemodelinspectordialog.cpp \ cppcodemodelinspectordialog.cpp \
cppdocumentationcommenthelper.cpp \ cppdocumentationcommenthelper.cpp \
cppeditor.cpp \ cppeditor.cpp \

View File

@@ -22,8 +22,6 @@ QtcPlugin {
files: [ files: [
"cppautocompleter.cpp", "cppautocompleter.cpp",
"cppautocompleter.h", "cppautocompleter.h",
"cppcanonicalsymbol.cpp",
"cppcanonicalsymbol.h",
"cppcodemodelinspectordialog.cpp", "cppcodemodelinspectordialog.cpp",
"cppcodemodelinspectordialog.h", "cppcodemodelinspectordialog.h",
"cppcodemodelinspectordialog.ui", "cppcodemodelinspectordialog.ui",

View File

@@ -25,9 +25,9 @@
#include "cppuseselectionsupdater.h" #include "cppuseselectionsupdater.h"
#include "cppcanonicalsymbol.h"
#include "cppeditor.h" #include "cppeditor.h"
#include <cpptools/cppcanonicalsymbol.h>
#include <cpptools/cpplocalsymbols.h> #include <cpptools/cpplocalsymbols.h>
#include <cpptools/cppmodelmanager.h> #include <cpptools/cppmodelmanager.h>
#include <cpptools/cpptoolsreuse.h> #include <cpptools/cpptoolsreuse.h>
@@ -125,7 +125,7 @@ public:
{ {
TextEditor::Convenience::convertPosition(textCursor.document(), textCursor.position(), TextEditor::Convenience::convertPosition(textCursor.document(), textCursor.position(),
&line, &column); &line, &column);
CppEditor::Internal::CanonicalSymbol canonicalSymbol(document, snapshot); CppTools::CanonicalSymbol canonicalSymbol(document, snapshot);
scope = canonicalSymbol.getScopeAndExpression(textCursor, &expression); scope = canonicalSymbol.getScopeAndExpression(textCursor, &expression);
} }
@@ -199,7 +199,7 @@ QList<int> findReferences(const Params &p)
typeOfExpression.init(p.document, snapshot); typeOfExpression.init(p.document, snapshot);
typeOfExpression.setExpandTemplates(true); typeOfExpression.setExpandTemplates(true);
using CppEditor::Internal::CanonicalSymbol; using CppTools::CanonicalSymbol;
if (Symbol *s = CanonicalSymbol::canonicalSymbol(p.scope, p.expression, typeOfExpression)) { if (Symbol *s = CanonicalSymbol::canonicalSymbol(p.scope, p.expression, typeOfExpression)) {
CppTools::CppModelManager *mmi = CppTools::CppModelManager::instance(); CppTools::CppModelManager *mmi = CppTools::CppModelManager::instance();
result = mmi->references(s, typeOfExpression.context()); result = mmi->references(s, typeOfExpression.context());

View File

@@ -35,8 +35,7 @@
using namespace CPlusPlus; using namespace CPlusPlus;
namespace CppEditor { namespace CppTools {
namespace Internal {
CanonicalSymbol::CanonicalSymbol(const Document::Ptr &document, CanonicalSymbol::CanonicalSymbol(const Document::Ptr &document,
const Snapshot &snapshot) const Snapshot &snapshot)
@@ -129,5 +128,4 @@ Symbol *CanonicalSymbol::canonicalSymbol(Scope *scope, const QString &code,
return 0; return 0;
} }
} // namespace Internal
} // namespace CppEditor } // namespace CppEditor

View File

@@ -25,16 +25,17 @@
#pragma once #pragma once
#include "cpptools_global.h"
#include <cplusplus/LookupContext.h> #include <cplusplus/LookupContext.h>
#include <cplusplus/Symbol.h> #include <cplusplus/Symbol.h>
#include <cplusplus/TypeOfExpression.h> #include <cplusplus/TypeOfExpression.h>
QT_FORWARD_DECLARE_CLASS(QTextCursor) QT_FORWARD_DECLARE_CLASS(QTextCursor)
namespace CppEditor { namespace CppTools {
namespace Internal {
class CanonicalSymbol class CPPTOOLS_EXPORT CanonicalSymbol
{ {
public: public:
CanonicalSymbol(const CPlusPlus::Document::Ptr &document, CanonicalSymbol(const CPlusPlus::Document::Ptr &document,
@@ -58,5 +59,4 @@ private:
CPlusPlus::TypeOfExpression m_typeOfExpression; CPlusPlus::TypeOfExpression m_typeOfExpression;
}; };
} // namespace Internal } // namespace CppTools
} // namespace CppEditor

View File

@@ -12,6 +12,7 @@ HEADERS += \
clangdiagnosticconfig.h \ clangdiagnosticconfig.h \
clangdiagnosticconfigsmodel.h \ clangdiagnosticconfigsmodel.h \
clangdiagnosticconfigswidget.h \ clangdiagnosticconfigswidget.h \
cppcanonicalsymbol.h \
cppchecksymbols.h \ cppchecksymbols.h \
cppclassesfilter.h \ cppclassesfilter.h \
cppcodeformatter.h \ cppcodeformatter.h \
@@ -95,6 +96,7 @@ SOURCES += \
clangdiagnosticconfig.cpp \ clangdiagnosticconfig.cpp \
clangdiagnosticconfigsmodel.cpp \ clangdiagnosticconfigsmodel.cpp \
clangdiagnosticconfigswidget.cpp \ clangdiagnosticconfigswidget.cpp \
cppcanonicalsymbol.cpp \
cppchecksymbols.cpp \ cppchecksymbols.cpp \
cppclassesfilter.cpp \ cppclassesfilter.cpp \
cppcodeformatter.cpp \ cppcodeformatter.cpp \

View File

@@ -51,6 +51,8 @@ Project {
"clangdiagnosticconfigswidget.ui", "clangdiagnosticconfigswidget.ui",
"compileroptionsbuilder.cpp", "compileroptionsbuilder.cpp",
"compileroptionsbuilder.h", "compileroptionsbuilder.h",
"cppcanonicalsymbol.cpp",
"cppcanonicalsymbol.h",
"cppchecksymbols.cpp", "cppchecksymbols.cpp",
"cppchecksymbols.h", "cppchecksymbols.h",
"cppclassesfilter.cpp", "cppclassesfilter.cpp",