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

[Eugene Zelenko via Phabricator via cfe-commits]

Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/clang-doc/HTMLGenerator.cpp:21
 
 namespace {
 
----------------
Anonymous namespace is used for functions when LLVM Coding Guidelines tells using static for same purpose.


================
Comment at: clang-tools-extra/clang-doc/HTMLGenerator.cpp:31
+                   const llvm::StringMap<std::string> &Options) {
+  std::string OptionsText = "";
+  for (auto &Option : Options) {
----------------
You don't need to assign empty string. See [[ https://clang.llvm.org/extra/clang-tidy/checks/readability-redundant-string-init.html | readability-redundant-string-init ]].


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

https://reviews.llvm.org/D63663