From f2bec2fed324d7f6922d3e8b2ba3cf6537b00982 Mon Sep 17 00:00:00 2001 From: Raphael Schaller Date: Wed, 9 Mar 2022 20:02:14 +0100 Subject: [PATCH] remove m_touched --- tests/SelfTest/UsageTests/MatchersRanges.tests.cpp | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/tests/SelfTest/UsageTests/MatchersRanges.tests.cpp b/tests/SelfTest/UsageTests/MatchersRanges.tests.cpp index 50bba04ed7..1e4b252e3e 100644 --- a/tests/SelfTest/UsageTests/MatchersRanges.tests.cpp +++ b/tests/SelfTest/UsageTests/MatchersRanges.tests.cpp @@ -115,10 +115,9 @@ template struct with_mocked_iterator_access { std::vector m_elements; // use plain arrays to have nicer printouts with CHECK(...) - mutable std::unique_ptr m_touched; mutable std::unique_ptr m_derefed; - // We want to check which elements were touched when iterating, so + // We want to check which elements were dereferenced when iterating, so // we can check whether iterator-using code traverses range correctly template class basic_iterator { template @@ -147,10 +146,6 @@ template struct with_mocked_iterator_access { } basic_iterator& operator++() { ++m_origin_idx; - assert( m_origin_idx < m_origin->m_elements.size() + 1 && "Outside of valid alloc" ); - if ( m_origin_idx < m_origin->m_elements.size() ) { - m_origin->m_touched[m_origin_idx] = true; - } return *this; } basic_iterator operator++( int ) { @@ -174,7 +169,6 @@ template struct with_mocked_iterator_access { with_mocked_iterator_access( std::initializer_list init ): m_elements( init ), - m_touched( std::make_unique( m_elements.size() ) ), m_derefed( std::make_unique( m_elements.size() ) ) {} const_iterator begin() const { return { this, 0 }; }