[PATCH] D141000: [clang-tidy] Introduce HeaderFileExtensions and ImplementationFileExtensions options

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 8 08:55:46 PST 2023


carlosgalvezp added a comment.

Since we are at it - are we happy with the name "ImplementationFileExtensions"? I think "SourceFileExtensions" is more common in the literature, and it would look more visually pleasing in the config file since it has the same length as "HeaderFileExtensions".

  HeaderFileExtensions: ["h", "hh"]
  ImplementationFileExtensions: ["c", "cc"]

vs

  HeaderFileExtensions: ["h", "hh"]
  SourceFileExtensions: ["c", "cc"]


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141000



More information about the cfe-commits mailing list