Fix some sanitizer issues

ETLCPP · Feb 23, 2025 · 5e689c0 · 5e689c0
1 parent 109d45c
commit 5e689c0
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 12 deletions.
diff --git a/test/test_algorithm.cpp b/test/test_algorithm.cpp
@@ -2010,10 +2010,10 @@ namespace
       {
         std::shuffle(data.begin(), data.end(), urng);
 
-        std::forward_list<int> data1(data.begin(), data.end());
+        std::vector<int>       data1(data.begin(), data.end());
         std::forward_list<int> data2(data.begin(), data.end());
 
-        data1.sort();
+        std::sort(data1.begin(), data1.end());
         etl::selection_sort(data2.begin(), data2.end());
 
         bool is_same = std::equal(data1.begin(), data1.end(), data2.begin());

diff --git a/test/test_forward_list.cpp b/test/test_forward_list.cpp
@@ -1075,14 +1075,13 @@ namespace
     //*************************************************************************
     TEST_FIXTURE(SetupFixture, test_reverse_empty)
     {
-      CompareDataNDC compare_data;
       DataNDC data;
 
-      compare_data.reverse();
       data.reverse();
 
-      are_equal = std::equal(data.begin(), data.end(), compare_data.begin());
-      CHECK(are_equal);
+      CHECK_TRUE(data.empty());
+      CHECK_FALSE(data.full());
+      CHECK_EQUAL(0, data.size());
     }
 
     //*************************************************************************

diff --git a/test/test_forward_list_shared_pool.cpp b/test/test_forward_list_shared_pool.cpp
@@ -1478,20 +1478,20 @@ namespace
     //*************************************************************************
     TEST_FIXTURE(SetupFixture, test_reverse_empty)
     {
-      CompareDataNDC compare_data;
       PoolNDC2 pool;
       DataNDC data1(pool);
       DataNDC data2(pool);
 
-      compare_data.reverse();
       data1.reverse();
       data2.reverse();
 
-      are_equal = std::equal(data1.begin(), data1.end(), compare_data.begin());
-      CHECK(are_equal);
+      CHECK_TRUE(data1.empty());
+      CHECK_FALSE(data1.full());
+      CHECK_EQUAL(0, data1.size());
 
-      are_equal = std::equal(data1.begin(), data1.end(), compare_data.begin());
-      CHECK(are_equal);
+      CHECK_TRUE(data2.empty());
+      CHECK_FALSE(data2.full());
+      CHECK_EQUAL(0, data2.size());
     }
 
     //*************************************************************************