aprantl added inline comments.
================ Comment at: include/clang/AST/PrettyPrinter.h:227 + + /// \brief Use formatting compatible with ABI specification. It is necessary for + /// saving entities into debug tables which have to be compatible with ---------------- xgsa wrote: > aprantl wrote: > > Te \brief is redundant and can be omitted. > I reviewed the other descriptions once again and I suppose it would be more > consistent to have "\brief Use formatting compatible with ABI specification." > and the rest of the description as a separate paragraph. Don't you mind > against such fix? This is obviously not super important, but since you asked: our policy for how we use Doxygen has evolved over time, and this file represents an older version of that policy. The right thing to do would be to first commit a cleanup patch that removes all existing `\brief` occurrences from the file and then land the patch. Repository: rL LLVM https://reviews.llvm.org/D39622 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
