CaseyCarter accepted this revision. CaseyCarter added a comment. LGTM!
================ Comment at: libcxx/include/iterator:1052 class _LIBCPP_TEMPLATE_VIS ostream_iterator : public iterator<output_iterator_tag, void, void, void, void> { ---------------- `std::iterator` has been deprecated since C++17. Should this inheritance be dependent on `_LIBCPP_STD_VER < 17`? It's unnecessary now that the member types are defined directly. (Also on line 1161.) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87459/new/ https://reviews.llvm.org/D87459 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits