[PATCH] D74669: [clang-tidy] New check: bugprone-suspicious-include

[Jonathan Roelofs via Phabricator via cfe-commits]

jroelofs marked 2 inline comments as done.
jroelofs added a comment.

In D74669#1902107 <https://reviews.llvm.org/D74669#1902107>, @njames93 wrote:

> Adding the parent revision means you don't need to have those changes in this patch.


IIUC, that is what I've already done:

https://reviews.llvm.org/D74669 is `git show c4dd6f5903e -U999`
https://reviews.llvm.org/D75489 is `git show 0cda7e0b9ed -U999`

where `c4dd6f5903e` is the parent of `0cda7e0b9ed`.

The suggestion to switch over to `;`s as delimiters is what's causing this diff to still touch so many files.



================
Comment at: clang-tools-extra/clang-tidy/bugprone/DynamicStaticInitializersCheck.h:24
+///     extensions of header files (The filename extensions should not contain
+///     "." prefix). "h;hh;hpp;hxx" by default.
+//
----------------
";h;hh;hpp;hxx"


================
Comment at: clang-tools-extra/clang-tidy/google/UnnamedNamespaceInHeaderCheck.h:25
+///     extensions of header files (The filename extensions should not contain
+///     "." prefix). "h;hh;hpp;hxx" by default.
+///
----------------
";h;hh;hpp;hxx"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74669