[PATCH] Implement std::experimental::sample
Marshall Clow
mclow.lists at gmail.com
Mon Apr 27 16:34:57 PDT 2015
I'm very confused here.
The LFTS document references N3925 as the source of the `sample` proposal.
That paper has sample code, and talks about both "reservoir sampling" and "selection sampling", and using one for random access iterators and the other for forward iterators, and includes a reference implementation. This code looks nothing like that.
I also don't see any tests that deal with:
1. Stability "Stable if and only if PopulationIterator meets the requirements of a ForwardIterator type."
2. Empty input ranges. (should write nothing)
3. test for n == 0 (should write nothing)
4. No tests with different kinds of iterators.
http://reviews.llvm.org/D9044
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list