Eugene.Zelenko added inline comments.
================ Comment at: clang-tidy/misc/IncorrectPointerCastCheck.cpp:80 + } +} // namespace misc + ---------------- Somehow, misc namespace is closed twice. ================ Comment at: docs/ReleaseNotes.rst:60 +- New :doc:`misc-incorrect-pointer-cast + <clang-tidy/checks/misc-incorrect-pointer-cast>` check ---------------- Will be good idea to rebase from trunk and use alphabetical order. ================ Comment at: docs/ReleaseNotes.rst:63 + + Warn for cases when pointer is cast and the pointed to type is + incompatible with allocated memory area type. This may lead to access memory ---------------- Warns. Please also use as much as possible from 80 characters. ================ Comment at: docs/clang-tidy/checks/misc-incorrect-pointer-cast.rst:10 + +Warn for cases when the pointed to type is wider than the allocated type. +For example char vs integer, long vs char etc. ---------------- Looks like two paragraphs are almost same. Will be good idea to rephrase. Please make sure that lines are no longer then 80 characters. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D48866 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
