[PATCH] D141133: [clang-tidy] Implement CppCoreGuideline F.54

Chris Cotter via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 30 06:06:49 PST 2023


ccotter marked 2 inline comments as done.
ccotter added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/cppcoreguidelines/AvoidCaptureDefaultWhenCapturingThisCheck.cpp:81
+          Lambda->getCaptureDefaultLoc(),
+          "lambdas that capture 'this' should not specify a capture default");
+
----------------
carlosgalvezp wrote:
> njames93 wrote:
> > Would be nice to show if `this` is implicitly captured.
> Was this comment addressed? I see it marked as "Not Done".
oh yes, I forgot to mark addressed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141133



More information about the cfe-commits mailing list