ericniebler / range-v3

Range library for C++14/17/20, basis for C++20's std::ranges
Other
4.11k stars 440 forks source link

Isn't range-v3's views::join complaint with P2328R1 fixing? #1688

Open denchat opened 2 years ago

denchat commented 2 years ago

This compiles

    auto out = std::ranges::views::iota(1,6)
               | std::ranges::views::transform([](auto n){ return to_string(n); })
               | std::ranges::views::join
               | ranges::to<string>;

[LIVE]

This does not

    auto out2 = ranges::views::iota(1,6)
                | ranges::views::transform([](auto n){ return to_string(n); })
                | ranges::views::join
                | ranges::to<string>;

[LIVE]


brevzin commented 2 years ago

This doesn't work in range-v3 because range-v3 simply doesn't implement that paper yet (see my PR I just linked to)

zeyangl commented 2 years ago

Yeah this one is important. Banging my head an hour trying to get it to work, just to realize it simply doesn't