[PATCH] D79456: [clangd] Complete filenames after < / ".

2020-05-19 Thread Sam McCall via Phabricator via cfe-commits
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

[PATCH] D79456: [clangd] Complete filenames after < / ".

2020-05-18 Thread Sam McCall via Phabricator via cfe-commits
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

[PATCH] D79456: [clangd] Complete filenames after < / ".

2020-05-18 Thread Sam McCall via Phabricator via cfe-commits
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

[PATCH] D79456: [clangd] Complete filenames after < / ".

2020-05-06 Thread Adam Czachorowski via Phabricator via cfe-commits
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

[PATCH] D79456: [clangd] Complete filenames after < / ".

2020-05-05 Thread Sam McCall via Phabricator via cfe-commits
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