[libcxx-commits] [PATCH] D130321: [libc++][ranges] Implement `ranges::shuffle`.

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Jul 22 06:17:18 PDT 2022


ldionne accepted this revision.
ldionne added inline comments.
This revision is now accepted and ready to land.


================
Comment at: libcxx/test/std/algorithms/alg.modifying.operations/alg.random.shuffle/ranges_shuffle.pass.cpp:242
+
+  { // Complexity: Exactly `(last - first) - 1` swaps.
+    {
----------------
var-const wrote:
> The standard requirement that the number of swaps is "exact" instead of "at most" doesn't make sense to me. Is this a wording issue, or am I missing something?
It is somewhat unusual indeed -- I don't think anybody would complain for making fewer swaps than specified.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130321/new/

https://reviews.llvm.org/D130321



More information about the libcxx-commits mailing list