Apply reviewer suggestions

Author: winner245

libcxx/test/std/containers/sequences/vector.bool/assign_iter_iter.pass.cpp

@@ -8,8 +8,8 @@
 
 // <vector>
 
-// void assign(_InputIterator __first, _InputIterator __last);
-// void assign(_ForwardIterator __first, _ForwardIterator __last);
+// template <class InputIt>
+// constexpr void assign(InputIt first, InputIt last);
 
 #include <vector>
 #include <cassert>
@@ -26,19 +26,19 @@ TEST_CONSTEXPR_CXX20 bool tests() {
       v.assign(It(in.begin()), It(in.end()));
       assert(v == in);
     }
-    { // No reallocation: fit wintin current size
-      bool in[]                    = {false, true, false, true, true};
-      TEST_CONSTEXPR std::size_t N = sizeof(in) / sizeof(in[0]);
+    { // No reallocation: fit within current size
+      bool in[]     = {false, true, false, true, true};
+      std::size_t N = sizeof(in) / sizeof(in[0]);
       std::vector<bool> v(2 * N, false);
       using It = forward_iterator<bool*>;
       v.assign(It(in), It(in + N));
       assert(v.size() == N);
       for (std::size_t i = 0; i < N; ++i)
         assert(v[i] == in[i]);
     }
-    { // No reallocation: fit wintin spare space
-      bool in[]                    = {false, true, false, true, true};
-      TEST_CONSTEXPR std::size_t N = sizeof(in) / sizeof(in[0]);
+    { // No reallocation: fit within spare space
+      bool in[]     = {false, true, false, true, true};
+      std::size_t N = sizeof(in) / sizeof(in[0]);
       std::vector<bool> v(N / 2, false);
       v.reserve(N * 2);
       using It = forward_iterator<bool*>;
@@ -58,19 +58,19 @@ TEST_CONSTEXPR_CXX20 bool tests() {
       v.assign(It(in.begin()), It(in.end()));
       assert(v == in);
     }
-    { // No reallocation: fit wintin current size
-      bool in[]                    = {false, true, false, true, true};
-      TEST_CONSTEXPR std::size_t N = sizeof(in) / sizeof(in[0]);
+    { // No reallocation: fit within current size
+      bool in[]     = {false, true, false, true, true};
+      std::size_t N = sizeof(in) / sizeof(in[0]);
       std::vector<bool> v(2 * N, false);
       using It = cpp17_input_iterator<bool*>;
       v.assign(It(in), It(in + N));
       assert(v.size() == N);
       for (std::size_t i = 0; i < N; ++i)
         assert(v[i] == in[i]);
     }
-    { // No reallocation: fit wintin spare space
-      bool in[]                    = {false, true, false, true, true};
-      TEST_CONSTEXPR std::size_t N = sizeof(in) / sizeof(in[0]);
+    { // No reallocation: fit within spare space
+      bool in[]     = {false, true, false, true, true};
+      std::size_t N = sizeof(in) / sizeof(in[0]);
       std::vector<bool> v(N / 2, false);
       v.reserve(N * 2);
       using It = cpp17_input_iterator<bool*>;

libcxx/test/std/containers/sequences/vector.bool/assign_size_value.pass.cpp

@@ -8,7 +8,7 @@
 
 // <vector>
 
-// void assign(size_type __n, const value_type& __x);
+// void assign(size_type n, const value_type& x);
 
 #include <vector>
 #include <cassert>
@@ -18,24 +18,24 @@
 TEST_CONSTEXPR_CXX20 bool tests() {
   {   // Test with various cases where assign may or may not trigger reallocations
     { // Reallocation happens
-      TEST_CONSTEXPR std::size_t N = 128;
+      std::size_t N = 128;
       std::vector<bool> v(5, false);
       assert(v.capacity() < N);
       v.assign(N, true);
       assert(v.size() == N);
       for (std::size_t i = 0; i < N; ++i)
         assert(v[i] == true);
     }
-    { // No reallocation: fit wintin current size
-      TEST_CONSTEXPR std::size_t N = 5;
+    { // No reallocation: fit within current size
+      std::size_t N = 5;
       std::vector<bool> v(2 * N, false);
       v.assign(N, true);
       assert(v.size() == N);
       for (std::size_t i = 0; i < N; ++i)
         assert(v[i] == true);
     }
-    { // No reallocation: fit wintin spare space
-      TEST_CONSTEXPR std::size_t N = 5;
+    { // No reallocation: fit within spare space
+      std::size_t N = 5;
       std::vector<bool> v(N / 2, false);
       v.reserve(N * 2);
       v.assign(N, true);