N3644 tests for map/multimap/set/multiset

Howard Hinnant hhinnant at apple.com
Thu Aug 8 14:40:21 PDT 2013


On Aug 7, 2013, at 6:14 PM, Marshall Clow <mclow.lists at gmail.com> wrote:

> On Aug 7, 2013, at 12:08 PM, Howard Hinnant <hhinnant at apple.com> wrote:
> 
>> On Aug 6, 2013, at 2:39 PM, Marshall Clow <mclow.lists at gmail.com> wrote:
>> 
>>> The existing implementations seem to work, so I just added some tests.
>> 
>> Your tests are passing by accident.  __tree_iterator and __tree_const_iterator in <__tree> need treatment for their default actor.
> 
> Ok. So I fixed that
> 
>>> The general iterator tests for <map> appeared to be missing, so I added that, too.
>> 
>> Thanks, but this is covered in map/map.access/iterator.pass.cpp.
> 
> Moved them.
> 
> And I made the __tree_const_iterator () constructor noexcept 
> 
> Revised patch attached.

Looks great, thanks!  Please commit.

Howard




More information about the cfe-commits mailing list