futogergely marked an inline comment as done. futogergely added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/cert/ObsolescentFunctionsCheck.cpp:48 + // Matching functions with safe replacements in annex K. + auto FunctionNamesWithAnnexKReplacementMatcher = hasAnyName( + "::asctime", "::ctime", "::fopen", "::freopen", "::bsearch", "::fprintf", ---------------- whisperity wrote: > Is this ordering specific in any way? Is the rule listing them in this order? > If not, can we have them listed alphabetically, for easier search and > potential later change? done ================ Comment at: clang-tools-extra/docs/clang-tidy/checks/cert-msc24-c.rst:10 +For the listed functions, an alternative, more secure replacement is suggested, if available. +The checker heavily relies on the functions from annex K (Bounds-checking interfaces) of C11. + ---------------- whisperity wrote: > (And consistent capitalisation later.) done CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91000/new/ https://reviews.llvm.org/D91000 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
