[Libreoffice-commits] core.git: o3tl/qa
Miklos Vajna
vmiklos at collabora.co.uk
Thu Jun 2 07:08:54 UTC 2016
o3tl/qa/test-sorted_vector.cxx | 112 ++++++++++++++++++++---------------------
1 file changed, 56 insertions(+), 56 deletions(-)
New commits:
commit 59fe25a4240b6b456456779b63694de18827d6f7
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date: Thu Jun 2 08:04:17 2016 +0200
CppunitTest_o3tl_tests: fix loplugin:cppunitassertequals warnings in ...
... sorted_vector
Change-Id: Ie6c5f0e187228e2cf7769c62c86ff3cba80ab6ce
Reviewed-on: https://gerrit.libreoffice.org/25804
Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>
Tested-by: Jenkins <ci at libreoffice.org>
diff --git a/o3tl/qa/test-sorted_vector.cxx b/o3tl/qa/test-sorted_vector.cxx
index 2253ced..471ca7e 100644
--- a/o3tl/qa/test-sorted_vector.cxx
+++ b/o3tl/qa/test-sorted_vector.cxx
@@ -45,27 +45,27 @@ public:
CPPUNIT_ASSERT( aVec.insert(p1).second );
CPPUNIT_ASSERT( !aVec.insert(p3).second );
- CPPUNIT_ASSERT( aVec.size() == 2 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(2), aVec.size() );
- CPPUNIT_ASSERT( aVec[0] == p1 );
- CPPUNIT_ASSERT( aVec[1] == p3 );
+ CPPUNIT_ASSERT_EQUAL( p1, aVec[0] );
+ CPPUNIT_ASSERT_EQUAL( p3, aVec[1] );
- CPPUNIT_ASSERT( *aVec.begin() == p1 );
- CPPUNIT_ASSERT( *(aVec.end()-1) == p3 );
+ CPPUNIT_ASSERT_EQUAL( p1, *aVec.begin() );
+ CPPUNIT_ASSERT_EQUAL( p3, *(aVec.end()-1) );
- CPPUNIT_ASSERT( aVec.front() == p1 );
- CPPUNIT_ASSERT( aVec.back() == p3 );
+ CPPUNIT_ASSERT_EQUAL( p1, aVec.front() );
+ CPPUNIT_ASSERT_EQUAL( p3, aVec.back() );
CPPUNIT_ASSERT( aVec.find(p1) != aVec.end() );
- CPPUNIT_ASSERT( aVec.find(p1) - aVec.begin() == 0 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<std::ptrdiff_t>(0), aVec.find(p1) - aVec.begin() );
CPPUNIT_ASSERT( aVec.find(p3) != aVec.end() );
- CPPUNIT_ASSERT( aVec.find(p3) - aVec.begin() == 1 );
- CPPUNIT_ASSERT( aVec.find(p2) == aVec.end() );
- CPPUNIT_ASSERT( aVec.find(p4) == aVec.end() );
+ CPPUNIT_ASSERT_EQUAL( static_cast<std::ptrdiff_t>(1), aVec.find(p3) - aVec.begin() );
+ CPPUNIT_ASSERT( bool(aVec.find(p2) == aVec.end()) );
+ CPPUNIT_ASSERT( bool(aVec.find(p4) == aVec.end()) );
- CPPUNIT_ASSERT( aVec.erase(p1) == 1 );
- CPPUNIT_ASSERT( aVec.size() == 1 );
- CPPUNIT_ASSERT( aVec.erase(p2) == 0 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(1), aVec.erase(p1) );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(1), aVec.size() );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(0), aVec.erase(p2) );
aVec.DeleteAndDestroyAll();
delete p1;
@@ -85,22 +85,22 @@ public:
aVec.insert(p2);
aVec.insert(p3);
- CPPUNIT_ASSERT( aVec.erase(p1) == 1 );
- CPPUNIT_ASSERT( aVec.size() == 2 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(1), aVec.erase(p1) );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(2), aVec.size() );
aVec.erase(1);
- CPPUNIT_ASSERT( aVec.size() == 1 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(1), aVec.size() );
- CPPUNIT_ASSERT( aVec.erase(p4) == 0 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(0), aVec.erase(p4) );
aVec.clear();
- CPPUNIT_ASSERT( aVec.size() == 0 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(0), aVec.size() );
aVec.insert(p1);
aVec.insert(p2);
aVec.insert(p3);
aVec.DeleteAndDestroyAll();
- CPPUNIT_ASSERT( aVec.size() == 0 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(0), aVec.size() );
delete p4;
}
@@ -118,7 +118,7 @@ public:
o3tl::sorted_vector<SwContent*, o3tl::less_ptr_to<SwContent> > aVec2;
aVec2.insert( aVec1 );
- CPPUNIT_ASSERT( aVec2.size() == 3 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(3), aVec2.size() );
delete p1;
delete p2;
delete p3;
@@ -136,8 +136,8 @@ public:
aVec.insert(p2);
aVec.insert(p3);
- CPPUNIT_ASSERT( aVec.lower_bound(p1) == aVec.begin() );
- CPPUNIT_ASSERT( aVec.lower_bound(p4) == aVec.end() );
+ CPPUNIT_ASSERT( bool(aVec.lower_bound(p1) == aVec.begin()) );
+ CPPUNIT_ASSERT( bool(aVec.lower_bound(p4) == aVec.end()) );
delete p1;
delete p2;
delete p3;
@@ -160,29 +160,29 @@ public:
CPPUNIT_ASSERT( aVec.insert(p1).second );
CPPUNIT_ASSERT( !aVec.insert(p3).second );
- CPPUNIT_ASSERT( aVec.size() == 2 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(2), aVec.size() );
- CPPUNIT_ASSERT( aVec[0] == p1 );
- CPPUNIT_ASSERT( aVec[1] == p3 );
+ CPPUNIT_ASSERT_EQUAL( p1, aVec[0] );
+ CPPUNIT_ASSERT_EQUAL( p3, aVec[1] );
CPPUNIT_ASSERT( aVec.insert(p2_2).second );
CPPUNIT_ASSERT( aVec.insert(p2_3).second );
CPPUNIT_ASSERT( !aVec.insert(p2_2).second );
CPPUNIT_ASSERT( aVec.insert(p2_4).second );
- CPPUNIT_ASSERT( aVec.size() == 5 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(5), aVec.size() );
- CPPUNIT_ASSERT( *aVec.begin() == p1 );
- CPPUNIT_ASSERT( *(aVec.end()-1) == p3 );
+ CPPUNIT_ASSERT_EQUAL( p1, *aVec.begin() );
+ CPPUNIT_ASSERT_EQUAL( p3, *(aVec.end()-1) );
- CPPUNIT_ASSERT( aVec.front() == p1 );
- CPPUNIT_ASSERT( aVec.back() == p3 );
+ CPPUNIT_ASSERT_EQUAL( p1, aVec.front() );
+ CPPUNIT_ASSERT_EQUAL( p3, aVec.back() );
CPPUNIT_ASSERT( aVec.find(p1) != aVec.end() );
- CPPUNIT_ASSERT( aVec.find(p1) - aVec.begin() == 0 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<std::ptrdiff_t>(0), aVec.find(p1) - aVec.begin() );
CPPUNIT_ASSERT( aVec.find(p3) != aVec.end() );
- CPPUNIT_ASSERT( aVec.find(p3) - aVec.begin() == 4 );
- CPPUNIT_ASSERT( aVec.find(p2) == aVec.end() );
- CPPUNIT_ASSERT( aVec.find(p4) == aVec.end() );
+ CPPUNIT_ASSERT_EQUAL( static_cast<std::ptrdiff_t>(4), aVec.find(p3) - aVec.begin() );
+ CPPUNIT_ASSERT( bool(aVec.find(p2) == aVec.end()) );
+ CPPUNIT_ASSERT( bool(aVec.find(p4) == aVec.end()) );
CPPUNIT_ASSERT( aVec.find(p2_2) != aVec.end() );
CPPUNIT_ASSERT( aVec.find(p2_2) - aVec.begin() >= 1 );
CPPUNIT_ASSERT( aVec.find(p2_2) - aVec.begin() < 4 );
@@ -193,11 +193,11 @@ public:
CPPUNIT_ASSERT( aVec.find(p2_4) - aVec.begin() >= 1 );
CPPUNIT_ASSERT( aVec.find(p2_4) - aVec.begin() < 4 );
- CPPUNIT_ASSERT( aVec.erase(p1) == 1 );
- CPPUNIT_ASSERT( aVec.size() == 4 );
- CPPUNIT_ASSERT( aVec.erase(p2) == 0 );
- CPPUNIT_ASSERT( aVec.erase(p2_3) == 1 );
- CPPUNIT_ASSERT( aVec.size() == 3 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(1), aVec.erase(p1) );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(4), aVec.size() );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(0), aVec.erase(p2) );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(1), aVec.erase(p2_3) );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(3), aVec.size() );
aVec.DeleteAndDestroyAll();
delete p1;
@@ -221,39 +221,39 @@ public:
aVec.insert(p2);
aVec.insert(p3);
- CPPUNIT_ASSERT( aVec.erase(p1) == 1 );
- CPPUNIT_ASSERT( aVec.size() == 2 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(1), aVec.erase(p1) );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(2), aVec.size() );
aVec.erase(1);
- CPPUNIT_ASSERT( aVec.size() == 1 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(1), aVec.size() );
- CPPUNIT_ASSERT( aVec.erase(p4) == 0 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(0), aVec.erase(p4) );
aVec.clear();
- CPPUNIT_ASSERT( aVec.size() == 0 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(0), aVec.size() );
aVec.insert(p1);
aVec.insert(p2);
aVec.insert(p3);
aVec.insert(p1_2);
- CPPUNIT_ASSERT( aVec.size() == 4 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(4), aVec.size() );
aVec.insert(p1_3);
- CPPUNIT_ASSERT( aVec.size() == 5 );
- CPPUNIT_ASSERT( aVec.erase(p1) == 1 );
- CPPUNIT_ASSERT( aVec.find(p1) == aVec.end() );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(5), aVec.size() );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(1), aVec.erase(p1) );
+ CPPUNIT_ASSERT( bool(aVec.find(p1) == aVec.end()) );
CPPUNIT_ASSERT( aVec.find(p1_2) != aVec.end() );
CPPUNIT_ASSERT( aVec.find(p1_3) != aVec.end() );
- CPPUNIT_ASSERT( aVec.erase(p1_3) == 1 );
- CPPUNIT_ASSERT( aVec.find(p1) == aVec.end() );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(1), aVec.erase(p1_3) );
+ CPPUNIT_ASSERT( bool(aVec.find(p1) == aVec.end()) );
CPPUNIT_ASSERT( aVec.find(p1_2) != aVec.end() );
- CPPUNIT_ASSERT( aVec.find(p1_3) == aVec.end() );
- CPPUNIT_ASSERT( aVec.erase(p1_3) == 0 );
- CPPUNIT_ASSERT( aVec.find(p1) == aVec.end() );
+ CPPUNIT_ASSERT( bool(aVec.find(p1_3) == aVec.end()) );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(0), aVec.erase(p1_3) );
+ CPPUNIT_ASSERT( bool(aVec.find(p1) == aVec.end()) );
CPPUNIT_ASSERT( aVec.find(p1_2) != aVec.end() );
- CPPUNIT_ASSERT( aVec.find(p1_3) == aVec.end() );
+ CPPUNIT_ASSERT( bool(aVec.find(p1_3) == aVec.end()) );
aVec.DeleteAndDestroyAll();
- CPPUNIT_ASSERT( aVec.size() == 0 );
+ CPPUNIT_ASSERT_EQUAL( static_cast<size_t>(0), aVec.size() );
delete p1;
delete p1_3;
delete p4;
More information about the Libreoffice-commits
mailing list