[PATCH] D43229: [clangd] Enable snippet completion based on client capabilities.

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 15 06:33:06 PST 2018


ilya-biryukov added a comment.

As discussed offline with @sammccall, we don't really need to hold `Optional<>` fields for configuration entries, as we only consume them in clangd and therefore can just set the default values explicitly.
Makes the code simpler and requires only one extra test (with snippets enabled).


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D43229





More information about the cfe-commits mailing list