This revision was automatically updated to reflect the committed changes.
sammccall marked an inline comment as done.
Closed by commit rG032727f4f839: [clangd] Complete filenames after < /
". (authored by sammccall).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://revie
sammccall marked 3 inline comments as done.
sammccall added a comment.
*finally* getting back to this...
Comment at: clang-tools-extra/clangd/CodeComplete.cpp:1922
+ Line.consume_front("include") || Line.consume_front("import") ||
+ Line.consume_front("import_next");
+ L
sammccall updated this revision to Diff 264721.
sammccall added a comment.
Address comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79456/new/
https://reviews.llvm.org/D79456
Files:
clang-tools-extra/clangd/ClangdLSPServer.cpp
clang-tool
adamcz added inline comments.
Comment at: clang-tools-extra/clangd/CodeComplete.cpp:1922
+ Line.consume_front("include") || Line.consume_front("import") ||
+ Line.consume_front("import_next");
+ Line = Line.ltrim();
Did you mean include_next?
Also, this w
sammccall created this revision.
sammccall added a reviewer: adamcz.
Herald added subscribers: cfe-commits, usaxena95, kadircet, arphaman, jkorous,
MaskRay, ilya-biryukov.
Herald added a project: clang.
Extract prefix filters to CodeComplete so it can be easily tested.
Fixes https://github.com/c