[libcxx-commits] [PATCH] D100275: [libcxx] adds `forward_iterator` and `forward_range`

[Michael Schellenberger Costa via Phabricator via libcxx-commits]

miscco accepted this revision.
miscco added inline comments.


================
Comment at: libcxx/test/std/iterators/iterator.requirements/iterator.concepts/iterator.concept.forward/forward_iterator.compile.pass.cpp:156
+
+static_assert(!std::forward_iterator<noncopyable_input_iterator>);
+
----------------
Feel free to disregard: That line gets drowned a bit. Should we add a banner above it? 


================
Comment at: libcxx/test/std/iterators/iterator.requirements/iterator.concepts/iterator.concept.forward/forward_iterator.compile.pass.cpp:158
+
+struct not_input_iterator {
+  using difference_type = std::ptrdiff_t;
----------------
I do not really like that name. We should say *why* it is not a input iterator without checking all the methods.

I think here it is `not_eq_comparable_iterator`. Ditto my wish for porting the iterator test machinery




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100275