hokein added a comment. Nice! looks mostly good to me.
================ Comment at: clang-tidy/google/FunctionNamingCheck.cpp:57 + functionDecl( + isDefinition(), + unless(anyOf(isMain(), matchesName(validFunctionNameRegex(true)), ---------------- any reason why we restrict to definitions only? I think we can consider declarations too. ================ Comment at: unittests/clang-tidy/GoogleModuleTest.cpp:109 +TEST(ObjCFunctionNaming, AllowedStaticFunctionName) { + std::vector<ClangTidyError> Errors; ---------------- nit: we don't need unittest for the check here, as it is well covered in the littest. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D51575 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits