[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