[PATCH] D42484: [clangd] Limit completion results.

[Haojian Wu via Phabricator via cfe-commits]

hokein added inline comments.


================
Comment at: clangd/tool/ClangdMain.cpp:131
+                   "0 means no limit."),
+    llvm::cl::init(100), llvm::cl::Hidden);
+
----------------
sammccall wrote:
> I think we probably want to keep the default at 0 for now. I'm fine with 100 personally, but talk to Ilya if you want to change the default :-)
> 
> This shouldn't be hidden.
Let's keep the original behavior at the moment.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D42484