[PATCH] D44088: [clangd] Extract ClangdServer::Options struct.

[Jan Korous via Phabricator via llvm-commits]

jkorous-apple added inline comments.


================
Comment at: clang-tools-extra/trunk/clangd/ClangdServer.h:131
+  // Features like indexing must be enabled if desired.
+  static Options optsForTest();
+
----------------
Shouldn't this be better implemented as a utility function in tests?

Also what is the purpose of making global function in header file static?




Repository:
  rL LLVM

https://reviews.llvm.org/D44088