[PATCH] D113472: [clang-tidy] Fix lint warning in ClangTidyDiagnosticConsumer.cpp (NFC)

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 9 03:53:34 PST 2021


carlosgalvezp added a comment.

> If it is to be robust with /* multiline comments */

Does it have to be? I don't recall that we have unit tests for that. I would personally restrict the usage to only one way to write the line. I think it's not hard for users to adapt to that, plus it encourage them to write more readable code.

But OK, since it can potentially be a large change let's leave it for another time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113472



More information about the cfe-commits mailing list