[PATCH] D126186: [clang-tidy] Extend cert-oop57-cpp to check non-zero memset values

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 23 03:20:36 PDT 2022


njames93 added a comment.

Please upload diff with full context.
Can you add a note about this in the release notes.

As a side point I'm not sure this change really follows what the rule is trying to before. The rule is about not using std::memset to reinitialise objects that aren't trivial. Having said that limiting it to only 0 does seem a little restrictive,


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126186/new/

https://reviews.llvm.org/D126186



More information about the cfe-commits mailing list