NoQ added inline comments.
================ Comment at: clang/lib/Analysis/PlistHTMLPathDiagnosticConsumer.cpp:25 + PathDiagnosticConsumerOptions DiagOpts, PathDiagnosticConsumers &C, + const std::string &prefix, const Preprocessor &PP, + const cross_tu::CrossTranslationUnitContext &CTU) { ---------------- vsavchenko wrote: > As long as it is a new function, I guess we should definitely keep names > according to the **Coding Standards** It's not new but sure! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67422/new/ https://reviews.llvm.org/D67422 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits