This is an archive of the discontinued LLVM Phabricator instance.

Differential D107365

clangd: Make documentation property of completion items more similar
ClosedPublic

Authored by ckandeler on Aug 3 2021, 9:01 AM.

Details

Reviewers
sammccall
Commits
rG6b28835b3754: clangd: Make documentation property of completion items more similar
Summary

... to the one of signature hints.
In particular, completion items now also carry annotations, which client
code might be interested in.

Diff Detail

Event Timeline

ckandeler created this revision.Aug 3 2021, 9:01 AM
Herald added subscribers: usaxena95, kadircet, arphaman. · View Herald TranscriptAug 3 2021, 9:01 AM
ckandeler requested review of this revision.Aug 3 2021, 9:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 3 2021, 9:01 AM
Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. · View Herald Transcript
Harbormaster completed remote builds in B117660: Diff 363762.Aug 3 2021, 10:02 AM
ckandeler updated this revision to Diff 364412.Aug 5 2021, 4:24 AM

Addressed lint complaints.

Harbormaster completed remote builds in B118126: Diff 364412.Aug 5 2021, 5:32 AM
sammccall accepted this revision.Aug 6 2021, 5:46 AM

LG but could you please add a test showing this difference?

This revision is now accepted and ready to land.Aug 6 2021, 5:46 AM
ckandeler updated this revision to Diff 364790.Aug 6 2021, 7:29 AM

Added test coverage.

Harbormaster completed remote builds in B118380: Diff 364790.Aug 6 2021, 8:15 AM
This revision was landed with ongoing or failed builds.Aug 12 2021, 3:18 PM
Closed by commit rG6b28835b3754: clangd: Make documentation property of completion items more similar (authored by ckandeler, committed by sammccall). · Explain Why
This revision was automatically updated to reflect the committed changes.
sammccall added a commit: rG6b28835b3754: clangd: Make documentation property of completion items more similar.

Revision Contents

PathSize
clang-tools-extra/
clangd/
5 lines

Diff 363762

clang-tools-extra/clangd/CodeComplete.cpp

Show First 20 Lines • Show All 399 Lines • ▼ Show 20 Lines if (!Completion.Documentation) {
if (!Doc.empty()) { if (!Doc.empty()) {
Completion.Documentation.emplace(); Completion.Documentation.emplace();
parseDocumentation(Doc, *Completion.Documentation); parseDocumentation(Doc, *Completion.Documentation);
} }
}; };
if (C.IndexResult) { if (C.IndexResult) {
SetDoc(C.IndexResult->Documentation); SetDoc(C.IndexResult->Documentation);
} else if (C.SemaResult) { } else if (C.SemaResult) {
SetDoc(getDocComment(*ASTCtx, *C.SemaResult, const auto docComment = getDocComment(*ASTCtx, *C.SemaResult,
Lint: Pre-merge checks
Inline Actions

clang-tidy: warning: invalid case style for variable 'docComment' [readability-identifier-naming]
not useful

Lint: Pre-merge checks: clang-tidy: warning: invalid case style for variable 'docComment' [readability-identifier…
/*CommentsFromHeader=*/false)); /*CommentsFromHeader=*/false);
SetDoc(formatDocumentation(*SemaCCS, docComment));
} }
} }
} }
CodeCompletion build() { CodeCompletion build() {
Completion.ReturnType = summarizeReturnType(); Completion.ReturnType = summarizeReturnType();
Completion.Signature = summarizeSignature(); Completion.Signature = summarizeSignature();
Completion.SnippetSuffix = summarizeSnippet(); Completion.SnippetSuffix = summarizeSnippet();
▲ Show 20 Lines • Show All 1,640 Lines • Show Last 20 Lines