[PATCH] D79477: [clang-tidy] Add --use-color command line option and UseColor option to control colors in diagnostics
Nathan James via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu May 7 14:41:41 PDT 2020
njames93 added inline comments.
================
Comment at: clang-tools-extra/clang-tidy/ClangTidyOptions.cpp:99
IO.mapOptional("InheritParentConfig", Options.InheritParentConfig);
+ IO.mapOptional("UseColor", Options.UseColor);
}
----------------
I'm not entirely sure this option belongs in the config file. However if it will get read in here, maybe update the help string for the command line option to say it overrides the `UseColor` option from the `.clang-tidy` configuration file.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79477/new/
https://reviews.llvm.org/D79477
More information about the cfe-commits
mailing list