Re: Requesting endorsements of Boost.STLInterfaces for review

From: Gavin Lambert (boost_at_[hidden])
Date: 2019-08-23 03:14:24

On 23/08/2019 14:11, Zach Laine wrote:
> Online docs:
> 
> https://tzlaine.github.io/stl_inteTherfaces/doc/html/index.html
> <https://tzlaine.github.io/stl_interfaces/doc/html/index.html>

Speaking of iterators being error-prone (and the ironic typo in the 
above link):

In 
https://tzlaine.github.io/stl_interfaces/doc/html/boost_stlinterfaces__proposed_/introduction.html 
the long-form implementation of repeated_chars_iterator::operator>= is 
not correct.

And operator-(difference_type lhs, repeated_chars_iterator rhs) seems 
dubious; this operation is normally not commutative.  It seems more 
correct for that operator to not exist.  (And I hope that 
iterator_interface does not define it either.)

There's also several typos sprinkled around, such as "givenn" and using 
"if" instead of "it", etc.

I haven't read this in detail yet, but it sounds interesting.

Are you planning to add support for AllocatorAwareContainer at a later 
stage, or is there something that would make that infeasible?