[libcxx-commits] [PATCH] D144262: [libc++][ranges] Implement LWG-3865 Sorting a range of pairs

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added inline comments.


================
Comment at: libcxx/test/std/utilities/utility/pairs/pairs.spec/three_way_comparison.pass.cpp:39
+    assert((P1(2, 1) <=> P2(1, 2)) == std::strong_ordering::greater);
+    assert((P1(0, 0) <=> P2(0, 0)) == std::strong_ordering::equal);
+  }
----------------
fsb4000 wrote:
> Mordante wrote:
> > I would like to see a few more tests `int <=> double`, `int <=> string`.
> I added `int <=> double`
> 
> But what test do you want with `int <=> string` ?
> 
> 
Making sure it does not work, so this needs to be some `static_assert`.


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

https://reviews.llvm.org/D144262