diff --git a/src/plugins/cpptools/cppclassesfilter.cpp b/src/plugins/cpptools/cppclassesfilter.cpp index 62ea01f5f9a1535f3056bc8c38dd4587421fe683..1e516084136c15e2d95608c57d684db06c07c167 100644 --- a/src/plugins/cpptools/cppclassesfilter.cpp +++ b/src/plugins/cpptools/cppclassesfilter.cpp @@ -47,14 +47,6 @@ CppClassesFilter::~CppClassesFilter() { } -QString CppClassesFilter::stringToMatchUserInputAgainst(const ModelItemInfo &info, - bool userInputContainsColonColon) -{ - if (userInputContainsColonColon) - return info.scopedSymbolName(); - return info.symbolName; -} - Locator::FilterEntry CppClassesFilter::filterEntryFromModelItemInfo(const ModelItemInfo &info) { const QVariant id = qVariantFromValue(info); diff --git a/src/plugins/cpptools/cppclassesfilter.h b/src/plugins/cpptools/cppclassesfilter.h index 7f82d3bef8fbd3effc1452aa206553637e9e8246..b8d9bc39b330ab7d829689956132a926259a0b54 100644 --- a/src/plugins/cpptools/cppclassesfilter.h +++ b/src/plugins/cpptools/cppclassesfilter.h @@ -44,8 +44,6 @@ public: ~CppClassesFilter(); private: - QString stringToMatchUserInputAgainst(const ModelItemInfo &info, - bool userInputContainsColonColon); Locator::FilterEntry filterEntryFromModelItemInfo(const ModelItemInfo &info); }; diff --git a/src/plugins/cpptools/cppfunctionsfilter.cpp b/src/plugins/cpptools/cppfunctionsfilter.cpp index a82c8e4b81a6a8c54565cc379427c638718a08db..101b133bcd1f48592b21c3c4bae554a73ac508a5 100644 --- a/src/plugins/cpptools/cppfunctionsfilter.cpp +++ b/src/plugins/cpptools/cppfunctionsfilter.cpp @@ -46,14 +46,6 @@ CppFunctionsFilter::~CppFunctionsFilter() { } -QString CppFunctionsFilter::stringToMatchUserInputAgainst(const CppTools::ModelItemInfo &info, - bool userInputContainsColonColon) -{ - if (userInputContainsColonColon) - return info.scopedSymbolName(); - return info.symbolName; -} - Locator::FilterEntry CppFunctionsFilter::filterEntryFromModelItemInfo(const CppTools::ModelItemInfo &info) { const QVariant id = qVariantFromValue(info); diff --git a/src/plugins/cpptools/cppfunctionsfilter.h b/src/plugins/cpptools/cppfunctionsfilter.h index 58252105230e4832ae7a88ef2347b6933a4eba40..aa4e97d88ff2d38a4fde89f576651ac35599f6d5 100644 --- a/src/plugins/cpptools/cppfunctionsfilter.h +++ b/src/plugins/cpptools/cppfunctionsfilter.h @@ -44,8 +44,6 @@ public: ~CppFunctionsFilter(); private: - QString stringToMatchUserInputAgainst(const ModelItemInfo &info, - bool userInputContainsColonColon); Locator::FilterEntry filterEntryFromModelItemInfo(const ModelItemInfo &info); }; diff --git a/src/plugins/cpptools/cpplocatorfilter.cpp b/src/plugins/cpptools/cpplocatorfilter.cpp index 0dd7745e7effb83f2054bec951d2ca7cd853d66e..52cb87695c8c02f65e985f936c4b40c1a8e257fa 100644 --- a/src/plugins/cpptools/cpplocatorfilter.cpp +++ b/src/plugins/cpptools/cpplocatorfilter.cpp @@ -108,18 +108,10 @@ void CppLocatorFilter::onAboutToRemoveFiles(const QStringList &files) m_searchList.remove(file); } -QString CppLocatorFilter::stringToMatchUserInputAgainst(const CppTools::ModelItemInfo &info, - bool userInputContainsColonColon) -{ - Q_UNUSED(userInputContainsColonColon) - return info.scopedSymbolName(); -} - Locator::FilterEntry CppLocatorFilter::filterEntryFromModelItemInfo(const CppTools::ModelItemInfo &info) { const QVariant id = qVariantFromValue(info); - const QString name = info.symbolScope.isEmpty() ? info.symbolName : info.scopedSymbolName(); - Locator::FilterEntry filterEntry(this, name, id, info.icon); + Locator::FilterEntry filterEntry(this, info.symbolName, id, info.icon); filterEntry.extraInfo = info.type == ModelItemInfo::Class || info.type == ModelItemInfo::Enum ? info.shortNativeFilePath() : info.symbolType; @@ -163,7 +155,7 @@ QList<Locator::FilterEntry> CppLocatorFilter::matchesFor(QFutureInterface<Locato const QList<ModelItemInfo> items = it.value(); foreach (const ModelItemInfo &info, items) { - const QString matchString = stringToMatchUserInputAgainst(info, hasColonColon); + const QString matchString = hasColonColon ? info.scopedSymbolName() : info.symbolName; if ((hasWildcard && regexp.exactMatch(matchString)) || (!hasWildcard && matcher.indexIn(matchString) != -1)) { const Locator::FilterEntry filterEntry = filterEntryFromModelItemInfo(info); diff --git a/src/plugins/cpptools/cpplocatorfilter.h b/src/plugins/cpptools/cpplocatorfilter.h index 50e5184ecca301f11ffb8e27faf69aeaf553a419..866327cdd777ce2fd7b401c96c8454f4a40316c4 100644 --- a/src/plugins/cpptools/cpplocatorfilter.h +++ b/src/plugins/cpptools/cpplocatorfilter.h @@ -63,8 +63,6 @@ private slots: void onAboutToRemoveFiles(const QStringList &files); private: - virtual QString stringToMatchUserInputAgainst(const ModelItemInfo &info, - bool userInputContainsColonColon); virtual Locator::FilterEntry filterEntryFromModelItemInfo(const ModelItemInfo &info); private: diff --git a/src/plugins/cpptools/cpplocatorfilter_test.cpp b/src/plugins/cpptools/cpplocatorfilter_test.cpp index 682b78cd5b2804ec945a2fe445a8cd3ea25fd416..14803411a513cad0740f2bc4ffb909c08463ee9a 100644 --- a/src/plugins/cpptools/cpplocatorfilter_test.cpp +++ b/src/plugins/cpptools/cpplocatorfilter_test.cpp @@ -214,20 +214,17 @@ void CppToolsPlugin::test_cpplocatorfilters_CppLocatorFilter_data() << _("my") << (QList<ResultData>() << ResultData(_("MyClass"), testFileShort) - << ResultData(_("MyClass::MyClass"), _("()")) - << ResultData(_("MyClass::function2"), _("(bool, int)")) + << ResultData(_("MyClass"), _("()")) + << ResultData(_("MyClass"), _("()")) + << ResultData(_("MyClass"), testFileShort) + << ResultData(_("MyClass"), _("()")) + << ResultData(_("MyClass"), testFileShort) + << ResultData(_("MyEnum"), testFileShort) << ResultData(_("MyEnum"), testFileShort) - << ResultData(_("MyNamespace::MyClass"), testFileShort) - << ResultData(_("MyNamespace::MyClass::MyClass"), _("()")) - << ResultData(_("MyNamespace::MyClass::function2"), _("(bool, int)")) - << ResultData(_("MyNamespace::MyEnum"), testFileShort) - << ResultData(_("MyNamespace::myFunction"), _("(bool, int)")) + << ResultData(_("MyEnum"), testFileShort) + << ResultData(_("myFunction"), _("(bool, int)")) + << ResultData(_("myFunction"), _("(bool, int)")) << ResultData(_("myFunction"), _("(bool, int)")) - << ResultData(_("<anonymous namespace>::MyClass"), testFileShort) - << ResultData(_("<anonymous namespace>::MyClass::MyClass"), _("()")) - << ResultData(_("<anonymous namespace>::MyClass::function2"), _("(bool, int)")) - << ResultData(_("<anonymous namespace>::MyEnum"), testFileShort) - << ResultData(_("<anonymous namespace>::myFunction"), _("(bool, int)")) ); }