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

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 29 08:03:30 PST 2015


alexfh 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:
> aaron.ballman wrote:
> > 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?
> I'd use either a regular link, or both a link and the redirect.
One interesting aspect to consider is the bingability of check names, e.g. http://www.bing.com/search?q=google-build-namespaces or http://www.google.com/search?q=misc-new-delete-overloads (apparently, this one doesn't work well with bing).

I'm not sure redirects will play well with it, but we can try. Inclusion of the checks in the index seems like the must though, if we want the checks to be searchable.


http://reviews.llvm.org/D15796





More information about the cfe-commits mailing list