aaron.ballman added a comment. Have you run this over any large code bases to see whether the check triggers in practice?
================ Comment at: docs/clang-tidy/checks/list.rst:95 fuchsia-default-arguments + fuchsia-header-anon-namespaces (redirects to google-build-namespaces) <fuchsia-header-anon-namespaces> fuchsia-multiple-inheritance ---------------- This change looks unrelated to the patch and should be handled separately. ================ Comment at: docs/clang-tidy/checks/readability-redundant-data-call.rst:6 + +This check suggests removing redundant `.data()` calls. + ---------------- What does it mean for a `data()` call to be redundant? I think the documentation needs to explain that a bit more. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D45702 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits