[PATCH] D72954: [clang-doc] Improving Markdown Output

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 30 12:10:24 PST 2020


phosek accepted this revision.
phosek added a comment.
This revision is now accepted and ready to land.

LGTM % one minor nit, do you want me to land this for you?



================
Comment at: clang-tools-extra/clang-doc/assets/index.js:34
   var Path = computeRelativePath(Ref.Path, CurrentDirectory);
-  Path = append(Path, Ref.Name + ".html")
+  if (Ref.RefType == "namespace") {
+    Path = append(Path, "index.html");
----------------
You could omit braces here as well (we've been following C/C++ style guide for JS).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72954/new/

https://reviews.llvm.org/D72954





More information about the llvm-commits mailing list