[PATCH] D35986: [clangd] Add ':' to completion trigger characters.

2017-07-31 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL309550: [clangd] Add ':' to completion trigger characters. (authored by ibiryukov). Changed prior to commit: https://reviews.llvm.org/D35986?vs=108621&id=108887#toc Repository: rL LLVM https://revie

[PATCH] D35986: [clangd] Add ':' to completion trigger characters.

2017-07-30 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir accepted this revision. krasimir added a comment. This revision is now accepted and ready to land. Looks good! https://reviews.llvm.org/D35986 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/lis

[PATCH] D35986: [clangd] Add ':' to completion trigger characters.

2017-07-28 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov created this revision. Without it we don't get completion requests from VSCode after nested name qualifiers (e.g. after 'std::'). https://reviews.llvm.org/D35986 Files: clangd/ClangdLSPServer.cpp Index: clangd/ClangdLSPServer.cpp ===