[PATCH] D27806: [clang-tidy] Add obvious-invalid-range
Piotr Padlewski via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 16 04:57:13 PST 2016
Prazek marked 2 inline comments as done.
Prazek added inline comments.
Comment at: clang-tidy/obvious/InvalidRangeCheck.cpp:20-36
+ "std::for_each; std::find; std::find_if; std::find_end; "
+ "std::find_first_of; std::adjacent_find; std::count; std::count_if;"
+ "std::mismatch; std::equal; std::search; std::copy; "
+ "std::copy_backward; std::swap_ranges; std::transform; std::replace"
+ "std::replace_if; std::replace_copy; std::replace_copy_if; std::fill; "
+ "std::fill_n; std::generate; std::generate_n; std::remove; std::remove_if"
+ "std::remove_copy; std::remove_copy_if; std::unique; std::unique_copy;"
> I would go with one per line. It will be much more diff-friendly this way. And also much easier to add stuff in the middle and maybe keep it sorted.
I don't expect this list to change in any way in the future, and it already take more than 20 lines. I don't want to put about 80 lines only to define the names
More information about the cfe-commits