DiegoAstiazaran created this revision. DiegoAstiazaran added a reviewer: juliehockett. DiegoAstiazaran added a project: clang-tools-extra. juliehockett accepted this revision. juliehockett added a comment. This revision is now accepted and ready to land.
LGTM (let's make sure we double check this next time) HTML requires posix-style paths. https://reviews.llvm.org/D65827 Files: clang-tools-extra/clang-doc/HTMLGenerator.cpp Index: clang-tools-extra/clang-doc/HTMLGenerator.cpp =================================================================== --- clang-tools-extra/clang-doc/HTMLGenerator.cpp +++ clang-tools-extra/clang-doc/HTMLGenerator.cpp @@ -256,6 +256,8 @@ auto ScriptNode = llvm::make_unique<TagNode>(HTMLTag::TAG_SCRIPT); SmallString<128> ScriptPath = computeRelativePath("", InfoPath); llvm::sys::path::append(ScriptPath, llvm::sys::path::filename(FilePath)); + // Paths in HTML must be in posix-style + llvm::sys::path::native(ScriptPath, llvm::sys::path::Style::posix); ScriptNode->Attributes.try_emplace("src", ScriptPath); Out.emplace_back(std::move(ScriptNode)); }
Index: clang-tools-extra/clang-doc/HTMLGenerator.cpp =================================================================== --- clang-tools-extra/clang-doc/HTMLGenerator.cpp +++ clang-tools-extra/clang-doc/HTMLGenerator.cpp @@ -256,6 +256,8 @@ auto ScriptNode = llvm::make_unique<TagNode>(HTMLTag::TAG_SCRIPT); SmallString<128> ScriptPath = computeRelativePath("", InfoPath); llvm::sys::path::append(ScriptPath, llvm::sys::path::filename(FilePath)); + // Paths in HTML must be in posix-style + llvm::sys::path::native(ScriptPath, llvm::sys::path::Style::posix); ScriptNode->Attributes.try_emplace("src", ScriptPath); Out.emplace_back(std::move(ScriptNode)); }
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits