[PATCH] D86671: [clang-tidy] Add new case type to check variables with Hungarian notation

Eugene Zelenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 23 06:41:03 PST 2020


Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp:614
+  if (CRD->isUnion())
+    return "";
+
----------------
Returning {} as default initializer will be better. Same in other places.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86671



More information about the llvm-commits mailing list