[clang-doc] Correct improper file paths in HTML output

[ilovepi]

In index.js the logic of the ternary operator was backwards, preventing
us from generating the correct file paths, or relative paths in the HTML
output.

[ilovepi]

• [clang-doc] Correct improper file paths in HTML output #132103 👈 (View in Graphite)
• [clang-doc][NFC] Remove unnecessary directory cleanup #132101
• [clang-doc] Avoid deref of invalid std::optional #131939
• [clang-doc] Enable MD tests in basic-project #131924
• [clang-doc] Add testing without --repository for HTML documentation #131894
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[llvmbot]

@llvm/pr-subscribers-clang-tools-extra

Author: Paul Kirth (ilovepi)

Changes

In index.js the logic of the ternary operator was backwards, preventing
us from generating the correct file paths, or relative paths in the HTML
output.

---

Full diff: https://github.com/llvm/llvm-project/pull/132103.diff

1 Files Affected:

• (modified) clang-tools-extra/clang-doc/assets/index.js (+2-2)

diff --git a/clang-tools-extra/clang-doc/assets/index.js b/clang-tools-extra/clang-doc/assets/index.js
index 6a223de66f84a..58a92f049f19f 100644
--- a/clang-tools-extra/clang-doc/assets/index.js
+++ b/clang-tools-extra/clang-doc/assets/index.js
@@ -2,8 +2,8 @@ function genLink(Ref) {
   // we treat the file paths different depending on if we're
   // serving via a http server or viewing from a local
   var Path = window.location.protocol.startsWith("file") ?
-      `${window.location.protocol}//${window.location.host}/${Ref.Path}` :
-      `${window.location.protocol}//${RootPath}/${Ref.Path}`;
+      `${window.location.protocol}//${RootPath}/${Ref.Path}` :
+      `${window.location.protocol}//${window.location.host}/${Ref.Path}`;
   if (Ref.RefType === "namespace") {
     Path = `${Path}/index.html`
   } else if (Ref.Path === "") {

[ilovepi]

Merge activity

• Mar 20, 5:02 PM EDT: A user started a stack merge that includes this pull request via Graphite.
• Mar 20, 5:09 PM EDT: Graphite rebased this pull request as part of a merge.
• Mar 20, 5:11 PM EDT: A user merged this pull request with Graphite.