caio/clice
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: unify include handling with add_link_by_location

Browse Source 
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
ykiko
2026-04-09 17:12:14 +08:00
parent b691ed1d06
commit d48236de9c
1 changed files with 6 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
src/feature/document_links.cpp
View File

@@ -40,21 +40,6 @@ auto document_links(CompilationUnitRef unit, PositionEncoding encoding)
offset + static_cast<std::uint32_t>(close_pos + 1));
};
links.reserve(directives.includes.size() + directives.has_includes.size());
for(const auto& include: directives.includes) {
auto [fid, range] = unit.decompose_range(include.filename_range);
if(fid != interested || !range.valid()) {
continue;
}
protocol::DocumentLink link{
.range = to_range(converter, range),
};
link.target = std::string(unit.file_path(include.fid));
links.push_back(std::move(link));
}
auto add_link_by_location = [&](clang::SourceLocation loc, llvm::StringRef target) {
auto [fid, offset] = unit.decompose_location(loc);
if(fid != interested || offset >= content.size()) {
@@ -69,6 +54,12 @@ auto document_links(CompilationUnitRef unit, PositionEncoding encoding)
links.push_back(std::move(link));
};
for(const auto& include: directives.includes) {
if(include.fid.isValid()) {
add_link_by_location(include.location, unit.file_path(include.fid));
}
}
for(const auto& has_include: directives.has_includes) {
if(has_include.fid.isValid()) {
add_link_by_location(has_include.location, unit.file_path(has_include.fid));