On 23/08/2019 17:10, Zach Laine wrote:
There's also several typos sprinkled around, such as "givenn" and using "if" instead of "it", etc.
I just ran a spell-check on the docs. I think I got all the typos. I did not notice the if/it confusion. Where was that?
container_interface: "because if has no insert()".
Are you planning to add support for AllocatorAwareContainer at a later stage, or is there something that would make that infeasible?
I hate allocators. I will not be adding support for allocator-aware containers.
Thanks for the feedback. Does this mean that you'd like to see a review?
In principle yes, though I don't think I've looked at it enough to warrant a formal endorsement. I don't really write new iterators or containers very often, and when I do, it's usually the allocators that are the pain point.