[libcxx-commits] [PATCH] D119619: [libc++][ranges] Implement `std::sortable`.

[Arthur O'Dwyer via Phabricator via libcxx-commits]

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


================
Comment at: libcxx/test/std/iterators/iterator.requirements/alg.req.sortable/sortable.compile.pass.cpp:38
+static_assert( std::indirect_strict_weak_order<CompInt, std::projected<Foo*, Proj>>);
+static_assert( std::sortable<Foo*, CompInt, Proj>);
+
----------------
I think it would be useful (in the sense of self-documentation) to add
```
static_assert(!std::sortable<Foo*, CompInt>);
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119619