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 +<https://wiki.sei.cmu.edu/confluence/display/c/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 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71973/new/ https://reviews.llvm.org/D71973 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
