[PATCH] D22725: [clang-tidy] Add check 'modernize-use-algorithm'
Jonas Devlieghere via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 3 12:22:27 PDT 2016
JDevlieghere updated this revision to Diff 66688.
JDevlieghere marked 21 inline comments as done.
JDevlieghere added a comment.
Addresses comments from Aaron Ballman
@aaron.ballman Thanks for the thorough review! Can you check whether the tests I added address your concerns? Could you also elaborate on the case with the C-function pointer? Unless I explicitly cast it to void* the compiler rejects will reject it as an argument to memcpy. Am I missing a case where this could go wrong? I still added it to the pointer arithmetic check though, just to be sure.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 16339 bytes
Desc: not available
More information about the cfe-commits