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

[Konstantin Varlamov via Phabricator via libcxx-commits]

var-const added inline comments.


================
Comment at: libcxx/test/std/algorithms/alg.modifying.operations/alg.random.shuffle/ranges_shuffle.pass.cpp:242
+
+  { // Complexity: Exactly `(last - first) - 1` swaps.
+    {
----------------
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?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130321