[PATCH] D113847: [clang-tidy][NFC] Simplify ClangTidyStats

Salman Javed via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 14 15:46:18 PST 2021


salman-javed-nz added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h:50
 struct ClangTidyStats {
-  ClangTidyStats()
-      : ErrorsDisplayed(0), ErrorsIgnoredCheckFilter(0), ErrorsIgnoredNOLINT(0),
-        ErrorsIgnoredNonUserCode(0), ErrorsIgnoredLineFilter(0) {}
+  int ErrorsDisplayed{0};
+  int ErrorsIgnoredCheckFilter{0};
----------------
What's the prevalent style for class member initialization? `=` or `{}`?
cppcoreguidelines-prefer-member-initializer defaults to `{}` but I have seen both types in the code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113847



More information about the cfe-commits mailing list