[PATCH] D15796: [PATCH] clang-tidy documentation redirects

[Aaron Ballman via cfe-commits]

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