simon.giesecke added inline comments.
================ Comment at: clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-prefer-member-initializer.cpp:537 +struct PR53515 { + int M; ---------------- Can this be renamed to something describing the test case? E.g. `AlreadyHasInitializer` (and the case above might be renamed to `AlreadyHasDefaultInitializer` to differentiate the two cases) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118927/new/ https://reviews.llvm.org/D118927 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits