[PATCH] D157057: [clang-tidy] Implement cppcoreguidelines CP.52

Chris Cotter via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 5 13:11:04 PDT 2023


ccotter added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/cppcoreguidelines/NoSuspendWithLockCheck.cpp:24-26
+                              hasType(recordDecl(hasAnyName(
+                                  "::std::unique_lock", "::std::lock_guard",
+                                  "::std::scoped_lock", "::std::shared_lock"))))
----------------
PiotrZSL wrote:
> add configuration option for lock types, many big project got own types or wrappers.
I was just thinking about this case...my code base had its own lock types that I would want to be able to include.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157057



More information about the cfe-commits mailing list