[PATCH] D65987: [clang-doc] Generate HTML links for children namespaces/records
Julie Hockett via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 12 10:52:21 PDT 2019
juliehockett accepted this revision.
juliehockett marked an inline comment as done.
juliehockett added a comment.
This revision is now accepted and ready to land.
Comment at: clang-tools-extra/clang-doc/HTMLGenerator.cpp:250
+ // The resulting path is "../../../A/B/D" instead of a "../D". It is correct
+ // but it would be better to have the shorter version.
StringRef Dir = Directory;
> DiegoAstiazaran wrote:
> > juliehockett wrote:
> > > Would `llvm::sys::path::remove_dots()` do this? It might not, but is worth investigating.
> > `llvm::sys::path::remove_dots()` removes all `../` (except for leading `../`) so the resulting path in the example would be `../A/B/D`, which is not correct.
> Sorry about that, I was incorrect. `llvm::sys::path::remove_dots()` do what we want but from the right side of the path, not the left side.
> For `A/B/C/../..` it changes it to `A`. So it still does not work for us.
That's irritating :(
CHANGES SINCE LAST ACTION
More information about the cfe-commits