aaron.ballman added inline comments. ================ Comment at: docs/clang-tidy/checks/cert-dcl54-cpp.rst:5 @@ +4,2 @@ +.. meta:: + :http-equiv=refresh: 0;URL=misc-new-delete-overloads.html ---------------- alexfh wrote: > Since this is not a deprecated/obsolete file, it might be a good idea to also > add a textual redirection with native RST links for media not supporting HTTP > redirects (e.g. PDF or whatever else can be generated from the RST files). So basically use a regular link instead of the meta redirect?
================ Comment at: docs/clang-tidy/checks/google-build-namespaces.rst:6 @@ -5,2 +5,3 @@ +Aliases: cert-dcl59-cpp ---------------- alexfh wrote: > nit: Missing trailing period. > > Also, it might be a good idea to make this slightly more verbose, e.g. > s/Aliases/This check is available under the following names/ or something > like this. Agreed. http://reviews.llvm.org/D15796 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
