baloghadamsoftware added inline comments. ================ Comment at: clang-tidy/misc/UnconventionalAssignOperatorCheck.cpp:69 @@ +68,3 @@ +void UnconventionalAssignOperatorCheck::check(const MatchFinder::MatchResult &Result) { + if (const auto *RetStmt = Result.Nodes.getNodeAs<ReturnStmt>("returnStmt")) { + diag(RetStmt->getLocStart(), "operator=() should always return '*this'"); ---------------- sbenza wrote: > couldn't this be folded into the Messages array below? There would be no benefit of it since we use the location of RetStmt in this message instead of Method.
http://reviews.llvm.org/D18265 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits