zinovy.nis added inline comments.
================ Comment at: clang-tidy/bugprone/ParentVirtualCallCheck.cpp:128 + diag(Member->getQualifierLoc().getSourceRange().getBegin(), + "'%0' is a grand-parent's method, not parent's. Did you mean %1?") + << CalleeName << ParentsStr; ---------------- aaron.ballman wrote: > The diagnostic should not contain punctuation aside from the question mark. > > Also, the diagnostic uses some novel terminology in "grandparent's method". > How about: "qualified function name %0 refers to a function that is not > defined by a direct base class; did you mean %1?" "is not defined by a direct base class" is not a correct proposal. Issues that this check finds are all about calling a grand-parent's method instead of parent's whether base class defines this method or not. How about > Qualified function name %0 refers to a function not from a direct base class; > did you mean %1? ? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D44295 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits