[PATCH] D71973: [clang-tidy] Add bugprone-suspicious-memory-comparison check

Gabor Bencze via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 2 16:13:13 PST 2020

gbencze marked 4 inline comments as done.
gbencze added a comment.

Thanks for all the feedback @JonasToth  :)

Comment at: clang-tools-extra/docs/clang-tidy/checks/bugprone-suspicious-memory-comparison.rst:10
+This check corresponds to the CERT C Coding Standard rule
+`EXP42-C. Do not compare padding data
JonasToth wrote:
> Maybe this link is not proper, because of the newline. could you please check if the documentation builds? (you need sphinx for that and enable it in cmake.)
It seems to work fine

Comment at: clang-tools-extra/docs/clang-tidy/checks/list.rst:75
    `bugprone-suspicious-enum-usage <bugprone-suspicious-enum-usage.html>`_,
+   `bugprone-suspicious-memory-comparison <bugprone-suspicious-memory-comparison.html>`_,
    `bugprone-suspicious-memset-usage <bugprone-suspicious-memset-usage.html>`_, "Yes"
JonasToth wrote:
> i believe the classification of the checks was changed today. did you rebase to master? hopefully it still applies clean.
I did rebase today and removed the classification for these two by hand



More information about the cfe-commits mailing list