[PATCH] D67545: [clang-tidy] Added DefaultOperatorNewCheck.

[Balázs Kéri via Phabricator via cfe-commits]

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