Use View instead of Iterator + size

[autoantwort]

No description provided.

[BillyONeal]

This is technically a loss of functionality, as the previous code works with arbitrary ranits but this requires contiguous. I'm strongly Neutral on that so going to ask other maintainers.

[BillyONeal]

Comparing to

vcpkg-tool/include/vcpkg/base/util.h

Line 75 in a50687c

void set_duplicates(FwdIt first, FwdIt last, OutIt out, Cmp cmp)

// the output range out.
template<class Container, class OutIt, class Cmp>
void set_duplicates(const Container& container, OutIt out, Cmp cmp)
{
    auto first = container.begin();
    const auto last = container.end();
    // as before

template<class Container, class OutIt>
void set_duplicates(const Container& container, OutIt out)
{
    return set_duplicates(container, out, std::less<>{});
}

Poll: We like this form over what's written?

@ras0219-msft Yes
@JavierMatosD Yes
@vicroms Yes
@BillyONeal No (but ate standard library code for breakfast for 5 years)
@data-queue No preference.

@ras0219-msft specifically asked for Container&& rather than View to avoid needing to name T at the call site.

[BillyONeal]

Thanks!