balazske marked an inline comment as done.
balazske 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>(
----------------
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.
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
