[PATCH] D90531: [clangd] Add clang-tidy options to config

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 6 17:28:57 PST 2020


njames93 updated this revision to Diff 303596.
njames93 marked an inline comment as done.
njames93 added a comment.

Removed Enable.
Removed the scalarBool parser as it was only needed by Enable.
Removed DynamicDictParser in favour of reworking DictParser to get better control of how to handle unknown keys.
Use a StringMap to store the CheckOptions in the Config.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90531/new/

https://reviews.llvm.org/D90531

Files:
  clang-tools-extra/clangd/Config.h
  clang-tools-extra/clangd/ConfigCompile.cpp
  clang-tools-extra/clangd/ConfigFragment.h
  clang-tools-extra/clangd/ConfigYAML.cpp
  clang-tools-extra/clangd/unittests/ConfigCompileTests.cpp
  clang-tools-extra/clangd/unittests/ConfigYAMLTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D90531.303596.patch
Type: text/x-patch
Size: 12236 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20201107/da5f60b0/attachment-0001.bin>


More information about the cfe-commits mailing list