feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

DocumentLocatorFilter: Make internals reusable in LocatorMatcher

Browse Source 
Prepare for LocatorMatcher implementation.

Change-Id: I3d78045a06c8ebefacb033f0eabb5f7a2457ab06
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
This commit is contained in:
Jarek Kobus
2023-04-17 18:02:55 +02:00
parent 90c09c65f9
commit 21c71cded7
3 changed files with 38 additions and 52 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/plugins/clangcodemodel/clangdlocatorfilters.cpp
View File

@@ -218,25 +218,23 @@ private:
};
QList<Entry> docEntries;
const auto docSymbolGenerator = [&](const DocumentSymbol &info,
const auto docSymbolModifier = [&docEntries](LocatorFilterEntry &entry,
const DocumentSymbol &info,
const LocatorFilterEntry &parent) {
LocatorFilterEntry entry;
entry.displayName = ClangdClient::displayNameFromDocumentSymbol(
static_cast<SymbolKind>(info.kind()), info.name(),
info.detail().value_or(QString()));
entry.linkForEditor = linkForDocSymbol(info);
entry.extraInfo = parent.extraInfo;
if (!entry.extraInfo.isEmpty())
entry.extraInfo.append("::");
entry.extraInfo.append(parent.displayName);
// TODO: Can we extend clangd to send visibility information?
entry.displayIcon = LanguageClient::symbolIcon(info.kind());
docEntries.append({entry, info});
return entry;
};
QList<LocatorFilterEntry> allMatches = matchesForImpl(future, entry, docSymbolGenerator);
const QList<LocatorFilterEntry> allMatches = matchesForImpl(future, entry,
docSymbolModifier);
if (docEntries.isEmpty())
return allMatches; // SymbolInformation case

57
src/plugins/languageclient/locatorfilter.cpp
View File

@@ -190,7 +190,7 @@ void DocumentLocatorFilter::resetSymbols()
m_currentSymbols.reset();
}
static LocatorFilterEntry generateLocatorEntry(const SymbolInformation &info,
static LocatorFilterEntry entryForSymbolInfo(const SymbolInformation &info,
DocumentUri::PathMapper pathMapper)
{
LocatorFilterEntry entry;
@@ -202,30 +202,37 @@ static LocatorFilterEntry generateLocatorEntry(const SymbolInformation &info,
return entry;
}
QList<LocatorFilterEntry> DocumentLocatorFilter::entriesForSymbolsInfo(
const QList<SymbolInformation> &infoList, const QRegularExpression &regexp)
LocatorFilterEntries entriesForSymbolsInfo(const QList<SymbolInformation> &infoList,
const QRegularExpression &regexp, const DocumentUri::PathMapper &pathMapper)
{
QTC_ASSERT(m_pathMapper, return {});
QList<LocatorFilterEntry> entries;
QTC_ASSERT(pathMapper, return {});
LocatorFilterEntries entries;
for (const SymbolInformation &info : infoList) {
if (regexp.match(info.name()).hasMatch())
entries << LanguageClient::generateLocatorEntry(info, m_pathMapper);
entries << LanguageClient::entryForSymbolInfo(info, pathMapper);
}
return entries;
}
QList<LocatorFilterEntry> DocumentLocatorFilter::entriesForDocSymbols(
const QList<DocumentSymbol> &infoList, const QRegularExpression &regexp,
const DocSymbolGenerator &docSymbolGenerator, const LocatorFilterEntry &parent)
LocatorFilterEntries entriesForDocSymbols(const QList<DocumentSymbol> &infoList,
const QRegularExpression &regexp, const FilePath &filePath,
const DocSymbolModifier &docSymbolModifier, const LocatorFilterEntry &parent = {})
{
QList<LocatorFilterEntry> entries;
LocatorFilterEntries entries;
for (const DocumentSymbol &info : infoList) {
const QList<DocumentSymbol> children = info.children().value_or(QList<DocumentSymbol>());
const bool hasMatch = regexp.match(info.name()).hasMatch();
const LocatorFilterEntry entry = hasMatch ? docSymbolGenerator(info, parent) : parent;
if (hasMatch)
LocatorFilterEntry entry;
if (hasMatch) {
entry.displayIcon = LanguageClient::symbolIcon(info.kind());
const Position &pos = info.range().start();
entry.linkForEditor = {filePath, pos.line() + 1, pos.character()};
docSymbolModifier(entry, info, parent);
entries << entry;
entries << entriesForDocSymbols(children, regexp, docSymbolGenerator, entry);
} else {
entry = parent;
}
entries << entriesForDocSymbols(children, regexp, filePath, docSymbolModifier, entry);
}
return entries;
}
@@ -243,29 +250,19 @@ void DocumentLocatorFilter::prepareSearch(const QString &/*entry*/)
QList<LocatorFilterEntry> DocumentLocatorFilter::matchesFor(
QFutureInterface<LocatorFilterEntry> &future, const QString &entry)
{
const auto docSymbolGenerator = [this](const DocumentSymbol &info,
const auto docSymbolModifier = [](LocatorFilterEntry &entry, const DocumentSymbol &info,
const LocatorFilterEntry &parent) {
Q_UNUSED(parent)
LocatorFilterEntry entry;
entry.displayName = info.name();
if (std::optional<QString> detail = info.detail())
entry.extraInfo = detail.value_or(QString());
entry.displayIcon = symbolIcon(info.kind());
entry.linkForEditor = linkForDocSymbol(info);
return entry;
entry.extraInfo = *detail;
};
return matchesForImpl(future, entry, docSymbolGenerator);
}
Link DocumentLocatorFilter::linkForDocSymbol(const DocumentSymbol &info) const
{
const Position &pos = info.range().start();
return {m_currentFilePath, pos.line() + 1, pos.character()};
return matchesForImpl(future, entry, docSymbolModifier);
}
QList<LocatorFilterEntry> DocumentLocatorFilter::matchesForImpl(
QFutureInterface<LocatorFilterEntry> &future, const QString &entry,
const DocSymbolGenerator &docSymbolGenerator)
const DocSymbolModifier &docSymbolModifier)
{
const FuzzyMatcher::CaseSensitivity caseSensitivity
= ILocatorFilter::caseSensitivity(entry) == Qt::CaseSensitive
@@ -293,9 +290,9 @@ QList<LocatorFilterEntry> DocumentLocatorFilter::matchesForImpl(
QTC_ASSERT(m_currentSymbols.has_value(), return {});
if (auto list = std::get_if<QList<DocumentSymbol>>(&*m_currentSymbols))
return entriesForDocSymbols(*list, regExp, docSymbolGenerator);
return entriesForDocSymbols(*list, regExp, m_currentFilePath, docSymbolModifier);
else if (auto list = std::get_if<QList<SymbolInformation>>(&*m_currentSymbols))
return entriesForSymbolsInfo(*list, regExp);
return entriesForSymbolsInfo(*list, regExp, m_pathMapper);
return {};
}
@@ -381,7 +378,7 @@ QList<LocatorFilterEntry> WorkspaceLocatorFilter::matchesFor(
});
}
auto generateEntry = [](const SymbolInfoWithPathMapper &info) {
return generateLocatorEntry(info.symbol, info.mapper);
return entryForSymbolInfo(info.symbol, info.mapper);
};
return Utils::transform(m_results, generateEntry).toList();
}

17
src/plugins/languageclient/locatorfilter.h
View File

@@ -19,6 +19,9 @@ namespace Core { class IEditor; }
namespace LanguageClient {
using DocSymbolModifier = std::function<void(Core::LocatorFilterEntry &,
const LanguageServerProtocol::DocumentSymbol &, const Core::LocatorFilterEntry &)>;
Core::LocatorMatcherTasks LANGUAGECLIENT_EXPORT workspaceMatchers(const QList<Client *> &clients,
Core::MatcherType type,
int maxResultCount = 0);
@@ -42,13 +45,9 @@ protected:
Utils::FilePath m_currentFilePath;
using DocSymbolGenerator = std::function<Core::LocatorFilterEntry(
const LanguageServerProtocol::DocumentSymbol &, const Core::LocatorFilterEntry &)>;
Utils::Link linkForDocSymbol(const LanguageServerProtocol::DocumentSymbol &info) const;
QList<Core::LocatorFilterEntry> matchesForImpl(
QFutureInterface<Core::LocatorFilterEntry> &future, const QString &entry,
const DocSymbolGenerator &docSymbolGenerator);
const DocSymbolModifier &docSymbolModifier);
private:
void updateCurrentClient();
@@ -56,14 +55,6 @@ private:
const LanguageServerProtocol::DocumentSymbolsResult &symbols);
void resetSymbols();
QList<Core::LocatorFilterEntry> entriesForSymbolsInfo(
const QList<LanguageServerProtocol::SymbolInformation> &infoList,
const QRegularExpression &regexp);
QList<Core::LocatorFilterEntry> entriesForDocSymbols(
const QList<LanguageServerProtocol::DocumentSymbol> &infoList,
const QRegularExpression &regexp, const DocSymbolGenerator &docSymbolGenerator,
const Core::LocatorFilterEntry &parent = {});
QMutex m_mutex;
QMetaObject::Connection m_updateSymbolsConnection;
QMetaObject::Connection m_resetSymbolsConnection;