Re: Interest in updated, standalone iterator_facade library?

From: Hans Dembinski (hans.dembinski_at_[hidden])
Date: 2019-08-12 08:41:22

Some more thoughts from a first glance at the library:

In https://tzlaine.github.io/iterator_facade/doc/html/boost_iteratorfacade__proposed_/tutorial.html
you list the user supplied operations. The method names should exactly correspond to the existing ones used by boost.iterator, see https://www.boost.org/doc/libs/1_71_0/libs/iterator/doc/iterator_facade.html

compare vs. distance_to
equals vs. equal
next vs. increment
prev vs. decrement

The reason is that the old names were likely already "optimised" by previous reviews and you should make it easy for people to port existing code to iterator2. Switching from iterator to iterator2 should be as easy changing the included header for simple cases.

Best regards,
Hans