malcolm.parsons added a comment. What happens if the function has `auto` as the return type?
================ Comment at: clang-tidy/modernize/ReturnBracedInitListCheck.cpp:23 +void ReturnBracedInitListCheck::registerMatchers(MatchFinder *Finder) { + // Only register the matchers for C++. + if (!getLangOpts().CPlusPlus) ---------------- C++11? ================ Comment at: clang-tidy/modernize/ReturnBracedInitListCheck.cpp:37 + functionDecl(isDefinition(), // Declarations don't have return statements. + returns(recordType()), // We onnly care about record types. + hasDescendant(returnStmt(hasDescendant(ctorAsArgument)))) ---------------- typo - only. Repository: rL LLVM https://reviews.llvm.org/D28768 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits