[PATCH] D91000: [clang-tidy] Add cert-msc24-c checker.

Fütő Gergely via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 3 04:01:34 PST 2021


futogergely marked an inline comment as done.
futogergely added a comment.

In D91000#3161296 <https://reviews.llvm.org/D91000#3161296>, @whisperity wrote:

> Should/does this work in C++ mode for `std::whatever`?

Right now the checker finds the functions in the global namespace only. The recommendation is listed only in the C part of the CERT rules, and as far as I know, Annex K functions are defined in the global namespace only, (or at least based on the standard). I can't really decide if the checker should look for the functions in the std namespace as well or not...


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

https://reviews.llvm.org/D91000



More information about the cfe-commits mailing list