Move from_range into details namespace

Author: 20162026

include/beman/inplace_vector/inplace_vector.hpp

@@ -37,15 +37,12 @@
 #endif
 #endif
 
-// beman::from_range_t
-namespace beman {
-struct from_range_t {};
-inline constexpr from_range_t from_range;
-}; // namespace beman
-
 // Private utilities
 namespace beman::inplace_vector::details {
 
+struct from_range_t {};
+inline constexpr from_range_t from_range;
+
 // clang-format off
 // Smallest unsigned integer that can represent values in [0, N].
 template <size_t N>
@@ -774,7 +771,7 @@ struct inplace_vector : public details::inplace_vector_base<T, N> {
   }
 
   template <details::container_compatible_range<T> R>
-  constexpr inplace_vector(beman::from_range_t, R &&rg)
+  constexpr inplace_vector(details::from_range_t, R &&rg)
     requires(std::constructible_from<T, std::ranges::range_reference_t<R>> &&
              std::movable<T>)
   {
@@ -918,7 +915,7 @@ struct inplace_vector : public details::inplace_vector_base<T, N> {
   = delete;
 
   template <details::container_compatible_range<T> R>
-  constexpr inplace_vector(beman::from_range_t, R &&rg)
+  constexpr inplace_vector(beman::inplace_vector::details::from_range_t, R &&rg)
     requires(std::constructible_from<T, std::ranges::range_reference_t<R>> &&
              std::movable<T>)
   = delete;

tests/beman/inplace_vector/constexpr.test.cpp

@@ -70,7 +70,7 @@ template <typename IV> constexpr void test_constructors() {
     IV v(arr.begin(), arr.end());
   }
   {
-    IV v(beman::from_range_t{}, arr);
+    IV v(beman::inplace_vector::details::from_range_t{}, arr);
   }
   {
     IV other{0, 1};

tests/beman/inplace_vector/constructors.test.cpp

@@ -115,21 +115,23 @@ TYPED_TEST(Constructors, CopyRanges) {
   auto reference = this->unique();
 
   {
-    IV device(beman::from_range, reference);
+    IV device(beman::inplace_vector::details::from_range, reference);
     EXPECT_EQ(device, reference);
   }
 
   if (IV::capacity() == 0)
     return;
 
   {
-    IV device(beman::from_range, reference | std::ranges::views::take(1));
+    IV device(beman::inplace_vector::details::from_range,
+              reference | std::ranges::views::take(1));
     EXPECT_EQ(device, IV{reference.front()});
   }
 
   {
     auto mid = std::midpoint(0ul, reference.size());
-    IV device(beman::from_range, reference | std::ranges::views::take(mid));
+    IV device(beman::inplace_vector::details::from_range,
+              reference | std::ranges::views::take(mid));
     EXPECT_EQ(device, IV(reference.begin(), reference.begin() + mid));
   }
 }

tests/beman/inplace_vector/freestanding.test.cpp

@@ -160,8 +160,11 @@ TEST(Freestanding, deleted) {
   static_assert(!std::is_constructible_v<FIV, input_iterator, input_iterator>);
 
   // constexpr inplace_vector(from_range_t, R&& rg);
-  static_assert(std::is_constructible_v<IV, beman::from_range_t, range>);
-  static_assert(!std::is_constructible_v<FIV, beman::from_range_t, range>);
+  static_assert(
+      std::is_constructible_v<IV, beman::inplace_vector::details::from_range_t,
+                              range>);
+  static_assert(!std::is_constructible_v<
+                FIV, beman::inplace_vector::details::from_range_t, range>);
 
   // constexpr inplace_vector(initializer_list<T> il);
   static_assert(std::is_constructible_v<IV, initializer_list>);