Switch to the new iterator idiom in all rules

[lucic71]

I also deleted the old PtrValueIter because it assumed that the inputs were random access iterators. This assumption is wrong for weaker iterator categories, for example bidirectional, see tests/unit/vector_iter_range_ctor.cpp:

// the iterator of map is bidirectional
std::map<int, int> m;
// the iterator of vector is random access, the the arguments (m.begin(), m.end())
// are bidirectional where PtrValueIter does not work
std::vector<std::pair<int, int>> v6(m.begin(), m.end());