balazske marked 8 inline comments as done.
balazske added a comment.
I was not sure how the aliasing is to be handled if the check is not exactly
the same as the original. (This was a topic on the mailing list.) I like it
more if only those checks are aliases that are exactly the same. But for now it
will remain as it was before, otherwise this has to be changed for all aliases.
================
Comment at: clang-tools-extra/clang-tidy/cert/CERTTidyModule.cpp:52
+// FIXME: The check can be improved to handle such cases.
+const llvm::StringRef CertErr33CCheckedFunctions = "::aligned_alloc;"
+ "::asctime_s;"
----------------
aaron.ballman wrote:
> Was this list generated automatically or manually? (Just wondering how hard
> to match it to the CERT rule -- spot checking hasn't found issues so far.)
I am not sure, I have the list for longer time, but it is almost sure that it
was generated by using text editor macros. The number of functions is correct
(with the excluded ones).
================
Comment at: clang-tools-extra/clang-tidy/cert/CERTTidyModule.cpp:328
Opts["cert-str34-c.DiagnoseSignedUnsignedCharComparisons"] = "false";
+ Opts["cert-err33-c.CheckedFunctions"] = CertErr33CCheckedFunctions;
return Options;
----------------
carlosgalvezp wrote:
> Same here
This is correct: 'str34' before 'err33'.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112409/new/
https://reviews.llvm.org/D112409
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
