Re: [PATCH] D13784: [clang-tidy] add check cppcoreguidelines-pro-type-union-access

[Samuel Benzaquen via cfe-commits]

sbenza added inline comments.

================
Comment at: clang-tidy/cppcoreguidelines/ProTypeUnionAccessCheck.cpp:20
@@ +19,3 @@
+void ProTypeUnionAccessCheck::registerMatchers(MatchFinder *Finder) {
+  
Finder->addMatcher(memberExpr(hasObjectExpression(hasType(recordDecl(isUnion())))).bind("expr"),
 this);
+}
----------------
This should check that we are in C++ code.

  if (!getLangOpts().CPlusPlus)
    return;


http://reviews.llvm.org/D13784



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits