[PATCH] D41326: [clang-tidy] Added diagnostics about incorrect usage of NOLINT comment

Anton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Dec 16 23:58:17 PST 2017


xgsa marked 3 inline comments as done.
xgsa added inline comments.


================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:339
     std::unique_ptr<ClangTidyOptionsProvider> OptionsProvider)
     : DiagEngine(nullptr), OptionsProvider(std::move(OptionsProvider)),
+      Profile(nullptr),
----------------
Eugene.Zelenko wrote:
> Please use default members initialization for DiagEngine and Profile.
Actually, it is not necessary, as unique_ptr is initialized with nullptr by default. I have just removed initializing them here.


https://reviews.llvm.org/D41326





More information about the cfe-commits mailing list