Recommend replacing postincrement with preincrement in any_of(), all_of(), non_of()

[spillner]

do {
if (!p(*first)
return false;
} while (++first != last);

would be more readable and sometimes more efficient (and perhaps even better-supported for some nonstandard iterator types.)

[mclow]

The problem with this code is that it does the wrong thing on empty collections; i.e, when last == first

[mclow]

To avoid postincrement, it might be better to use a degenerate for loop:

for ( ; first != last; ++first )
if ( !p (*first))
return false;
return true;

[mclow]

Changed in commit #4d598e84100b535fbb47bab16db313f4b1107287