[PATCH] D51214: [clangd] Add options to enable/disable fixits and function argument snippets.
Ilya Biryukov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 24 10:28:53 PDT 2018
ilya-biryukov added inline comments.
================
Comment at: clangd/tool/ClangdMain.cpp:197
+static llvm::cl::opt<bool> IncludeFixIts(
+ "include-fixits",
----------------
sammccall wrote:
> ilya-biryukov wrote:
> > I wonder if we should make the `IncludeFixIts` option hidden?
> > It currently only works for our YCM integration, VSCode and other clients break.
> why would a user want to turn this on or off?
Ideally, we want to have it always on.
However, all editors interpret the results we return in different ways. This is a temporary option until we can define how text edits are handled by LSP.
We filed the bugs, will dig them up on Monday.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D51214
More information about the cfe-commits
mailing list