alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land.
There's one more nit. Otherwise good to go. Thanks! ================ Comment at: clang/include/clang/Tooling/Core/Diagnostic.h:97 +/// Get the first fix to apply for this diagnostic. +/// Returns nullptr if no fixes are attached to the diagnostic. const llvm::StringMap<Replacements> *selectFirstFix(const Diagnostic& D); ---------------- This should be Doxygen's `\returns` keyword. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60857/new/ https://reviews.llvm.org/D60857 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits