[libcxx-commits] [PATCH] D102657: [libcxx][ranges] Update `{front_, back_, }insert_iterator` for C++20.

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue May 18 11:55:13 PDT 2021


Mordante added inline comments.


================
Comment at: libcxx/include/iterator:775
                       void,
                       void>
+#endif // _LIBCPP_STD_VER <= 17
----------------
It seems the class also got a default constructor in C++20. Can you add `_LIBCPP_INLINE_VISIBILITY constexpr back_insert_iterator() noexcept = default;`?
Please also update the synopsis and `front_insert_iterator`? With these changes I can use the `std::back_insert_iterator` in `std::format` :-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102657



More information about the libcxx-commits mailing list