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