juliehockett added inline comments.
================ Comment at: clang-tools-extra/clang-doc/Representation.h:385 + SourceRoot(SourceRoot), UserStylesheets(UserStylesheets), + JsScripts(JsScripts) { + if (!RepositoryUrl.empty()) { ---------------- Move to .cpp file (now that it has a non-trivial implementation) ================ Comment at: clang-tools-extra/clang-doc/Representation.h:396 + std::string OutDirectory; // Directory for outputting generated files. + std::string SourceRoot; // + llvm::Optional<std::string> RepositoryUrl; ---------------- Comment CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65483/new/ https://reviews.llvm.org/D65483 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits