Proposed resolution:

This wording is relative to N4910.

1. Modify [uninitialized.copy] as indicated:

   namespace ranges {
     template<input_iterator I, nothrow-forward-iterator O, nothrow-sentinel-for<O> S>
       requires constructible_from<iter_value_t<O>, iter_reference_t<I>>
       uninitialized_copy_n_result<I, O>
         uninitialized_copy_n(I ifirst, iter_difference_t<I> n, O ofirst, S olast);
   }
   

   -9- Preconditions: [ofirst, olast) does not overlap with ifirst + [0, n) .

   -10- Effects: Equivalent to:

   auto t = uninitialized_copy(counted_iterator(std::move(ifirst), n),
                               default_sentinel, ofirst, olast);
   return {std::move(t.in).base(), t.out};
   

2. Modify [uninitialized.move] as indicated:

   namespace ranges {
     template<input_iterator I, nothrow-forward-iterator O, nothrow-sentinel-for<O> S>
       requires constructible_from<iter_value_t<O>, iter_rvalue_reference_t<I>>
       uninitialized_move_n_result<I, O>
         uninitialized_move_n(I ifirst, iter_difference_t<I> n, O ofirst, S olast);
   }
   

   -8- Preconditions: [ofirst, olast) does not overlap with ifirst + [0, n) .

   -9- Effects: Equivalent to:

   auto t = uninitialized_move(counted_iterator(std::move(ifirst), n),
                               default_sentinel, ofirst, olast);
   return {std::move(t.in).base(), t.out};
   

3. Modify [specialized.destroy] as indicated:

   namespace ranges {
     template<nothrow-input-iterator I>
       requires destructible<iter_value_t<I>>
       constexpr I destroy_n(I first, iter_difference_t<I> n) noexcept;
   }
   

   -5- Effects: Equivalent to:

   return destroy(counted_iterator(std::move(first), n), default_sentinel).base();
   

[ 2022-11-12 Approved at November 2022 meeting in Kona. Status changed: Voting → WP. ]

[ 2022-08-23; Reflector poll ]

Set status to Tentatively Ready after eight votes in favour during reflector poll.

Currently, ranges::uninitialized_copy_n has the following equivalent Effects:

auto t = uninitialized_copy(counted_iterator(ifirst, n),
                            default_sentinel, ofirst, olast);
return {std::move(t.in).base(), t.out};

Given that ifirst is just an input_iterator which is not guaranteed to be copyable, we should move it into counted_iterator. The same goes for ranges::uninitialized_move_n and ranges::destroy_n.