[PATCH] D63663: [clang-doc] Add html links to the parents of a RecordInfo

Julie Hockett via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 21 12:40:59 PDT 2019


juliehockett added a comment.

General design thought: could we move the path name construction to `Serialize.cpp`? Essentially, move the logic that builds the path name from the namespaces and name to the serialize step (you'll have to add the `OutDirectory` to the `ClangDocContext` so that you can access it in the mapper), and then the `GetInfoOuputFile()` function in `ClangDocMain.cpp` would take the `Info.Path` string as a parameter and do the directory construction and whatnot.

With that, you could then also add the `Path` field to `Reference`s, so that you wouldn't need to pass a map around in the generation phase. What do you think?


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

https://reviews.llvm.org/D63663





More information about the cfe-commits mailing list