carlosgalvezp added inline comments.
================
Comment at:
clang-tools-extra/clang-tidy/readability/StaticAccessedThroughInstanceCheck.cpp:78
+ // Ignore anonymous structs/classes
+ if (StringRef(BaseTypeName).contains("(unnamed "))
+ return;
----------------
AMS21 wrote:
> I wonder if there's a cleaner and more performant way to check this.
> I've tried `isAnonymousStructOrUnion()` from the `RecordDecl` but that seems
> to be about something else.
Indeed this seems brittle. Since `RecordDecl` inherits from `NamedDecl`, have
you tried the `getName()` function? Hopefully it's an empty string and we can
use that instead?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147411/new/
https://reviews.llvm.org/D147411
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
