Adding assign(count, item) signature of assign. More to come to fin…

…ish #55 Fixing #57. Breaking change but I doubt anybody is using these ctor signatures.
OpenCyphal · Sep 6, 2023 · 8431cd0 · 8431cd0
1 parent 1ed0110
commit 8431cd0
Show file tree

Hide file tree

Showing 7 changed files with 1,137 additions and 954 deletions.
diff --git a/cetlvast/suites/docs/examples/example_08_variable_length_array_vs_vector.cpp b/cetlvast/suites/docs/examples/example_08_variable_length_array_vs_vector.cpp
@@ -83,8 +83,10 @@ TEST(example_08_variable_length_array_vs_vector, example_tight_fit_1)
     cetl::pf17::pmr::UnsynchronizedArrayMemoryResource<56>                        array_storage_1{};
     cetl::pf17::pmr::UnsynchronizedArrayMemoryResource<56>                        array_storage_0{&array_storage_1,
                                                                            array_storage_1.max_size()};
-    cetl::VariableLengthArray<char, cetl::pf17::pmr::polymorphic_allocator<char>> tight_fit{{&array_storage_0},
-                                                                                            array_storage_0.size()};
+    cetl::VariableLengthArray<char, cetl::pf17::pmr::polymorphic_allocator<char>> tight_fit{
+        array_storage_0.size(),
+        {&array_storage_0},
+    };
     for (std::size_t i = 0; i < 56; ++i)
     {
         std::cout << i << ", ";
@@ -102,8 +104,10 @@ TEST(example_08_variable_length_array_vs_vector, example_exact_fit)
     /// resource.
 
     cetl::pf17::pmr::UnsynchronizedArrayMemoryResource<56>                        array_storage_0{};
-    cetl::VariableLengthArray<char, cetl::pf17::pmr::polymorphic_allocator<char>> exact_fit{{&array_storage_0},
-                                                                                            array_storage_0.size()};
+    cetl::VariableLengthArray<char, cetl::pf17::pmr::polymorphic_allocator<char>> exact_fit{
+        array_storage_0.size(),
+        {&array_storage_0},
+    };
     exact_fit.reserve(56);
     for (std::size_t i = 0; i < 56; ++i)
     {

diff --git a/cetlvast/suites/unittest/test_variable_length_array_allocator_awareness.cpp b/cetlvast/suites/unittest/test_variable_length_array_allocator_awareness.cpp
diff --git a/cetlvast/suites/unittest/test_variable_length_array_bool.cpp b/cetlvast/suites/unittest/test_variable_length_array_bool.cpp
@@ -340,3 +340,27 @@ TYPED_TEST(VLABoolTests, TestBoolBack)
         const_cast<typename std::add_pointer<typename std::add_const<decltype(array)>::type>::type>(&array)->back();
     ASSERT_TRUE(value);
 }
+
+TYPED_TEST(VLABoolTests, TestAssignCountAndValue)
+{
+    auto array = TypeParam::make_bool_container();
+    array.assign(0, true);
+    ASSERT_EQ(0, array.size());
+    array.assign(1, false);
+    ASSERT_EQ(1, array.size());
+    ASSERT_FALSE(array[0]);
+    array.resize(9, false);
+    ASSERT_EQ(9, array.size());
+    array.assign(3, false);
+    ASSERT_EQ(3, array.size());
+    for(auto i = array.begin(), e = array.end(); i != e; ++i)
+    {
+        ASSERT_FALSE(*i);
+    }
+    array.assign(17, true);
+    ASSERT_EQ(17, array.size());
+    for(auto i = array.begin(), e = array.end(); i != e; ++i)
+    {
+        ASSERT_TRUE(*i);
+    }
+}