[PATCH] D63929: [clang-tidy] - Introduce abseil-prefixed-thread-annotations check.

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 28 06:55:16 PDT 2019


lebedev.ri added a comment.

(Stuck reviews are bad, but blind stamping is even worse..)



================
Comment at: clang-tools-extra/clang-tidy/abseil/PrefixedThreadAnnotationsCheck.cpp:15-16
+
+constexpr char kAbseilThreadAnnotationsHeader[] =
+    "absl/base/internal/thread_annotations.h";
+
----------------
constexpr StringLiteral ?
shouldn't this be in anonymous namespace, or be `static`?


================
Comment at: clang-tools-extra/clang-tidy/abseil/PrefixedThreadAnnotationsCheck.cpp:37
+    const MacroArgs *Args) {
+  llvm::StringRef actual_header =
+      SM.getFilename(MD.getMacroInfo()->getDefinitionLoc());
----------------
This isn't correct as per coding style


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D63929





More information about the cfe-commits mailing list