[PATCH] D83223: [clang-tidy] Header guard check can skip past license comment

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 6 12:54:03 PDT 2020


njames93 marked an inline comment as done.
njames93 added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/utils/HeaderGuard.cpp:313
         continue;
-
+      StringRef Opening = AddNewLine == NewLineInsertions::None ? "#ifndef "
+                          : AddNewLine == NewLineInsertions::One
----------------
clang-format(trunk) says what I submitted was correct, clang-format-10 is suggesting the change. Which do I trust??


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83223





More information about the cfe-commits mailing list