carlosgalvezp added a comment.

I found now what this global `User` option is about:

  /// Specifies the name or e-mail of the user running clang-tidy.
  ///
  /// This option is used, for example, to place the correct user name in TODO()
  /// comments in the relevant check.
  std::optional<std::string> User;

So it seems to me it's an option used only in one check, and yet it's a global 
option. Based on that I think it's reasonable to have `HeaderFileExtensions` 
and `ImplementationFileExtensions` (which are shared among multiple checks) as 
global as well. It doesn't need to be a CLI option, config-only is fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141000

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to