iterator_policies::sequence_fast_readonly not default constructible?

[zwhfly]

I found out that pybind11::detail::iterator_policies::sequence_fast_readonly is not default constructible.
This makes the type returned by pybind11::args::begin() not satisfying the std::input_or_output_iterator concept.

I added the default ctor directly in the header file: zwhfly@bfa326a, and made a test.

Given using Iter = decltype(std::declval<pybind11::args>().begin());, this change makes std::constructible_from<Iter> true, which makes std::default_initializable<Iter> true, which makes std::weakly_incrementable<Iter> true, which makes std::input_or_output_iterator<Iter> true, which makes pybind11::args satisfy std::ranges::range concept, which makes the following code work:

void f(pb11::args args)
{
    namespace views = std::ranges::views;
    auto vu = args | views::transform(
        [](pybind11::handle const & h) -> MyClass const & { return h.cast<MyClass const &>(); });
    some_library_function(vu.begin(), vu.end());
}

I'm not familiar with the source code of pybind11, so I wonder would this change cause any bad effect that I don't know of?
Could it be added?

[Skylion007]

@zwhfly Feel free to add. It seems like the use case could be compelling, just please be sure to add a test in your PR as well.

[Skylion007]

@zwhfly It actually appears as though the C++20 standard has been updated so that this is no longer an issue. See: https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2021/p2325r3.html