aaron.ballman added inline comments.
================
Comment at: clang-tools-extra/clang-tidy/cert/CERTTidyModule.cpp:53
+ CheckFactories.registerCheck<DefaultOperatorNewCheck>(
+ "cert-default-operator-new");
CheckFactories.registerCheck<performance::MoveConstructorInitCheck>(
----------------
balazske wrote:
> aaron.ballman wrote:
> > balazske wrote:
> > > The checker should be renamed to `cert-mem57-cpp` to comply with the
> > > others.
> > Yes, please. It should also be moved under a `// MEM` comment instead of
> > `OOP` (and kept in alphabetical order).
> I will move it, but should I change the order of the commented sections?
> These are currently not in alphabetic order.
Not for this patch, but that could be done in a separate patch (as an NFC
change).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67545/new/
https://reviews.llvm.org/D67545
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
