PiotrZSL added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp:367-379 + if (const auto *TD = dyn_cast<ValueDecl>(ND)) { + QualType QT = TD->getType(); + + size_t PtrCount = 0; + if (QT->isPointerType()) { + PtrCount = getAsteriskCount(Type); + } ---------------- this function is already big, extract this to have something like: ``` if (removeReudnantAsterisks(Type, ND)) { RedundantRemoved = true; break; } ``` ================ Comment at: clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp:696-702 + const std::string &TypeName) const { + size_t Pos = TypeName.find('*'); + size_t Count = 0; + for (; Pos < TypeName.length(); Pos++, Count++) { + if ('*' != TypeName[Pos]) + break; + } ---------------- what if type will be like std::optional<unsigned char**>, will this function also be executed ? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147779/new/ https://reviews.llvm.org/D147779 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits