[libcxx-commits] [PATCH] D142335: [libc++][ranges] Partially implement `ranges::to`.
Konstantin Varlamov via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Wed Apr 12 00:38:28 PDT 2023
var-const added inline comments.
================
Comment at: libcxx/include/__ranges/to.h:71
+template <class>
+concept __always_false = false;
+
----------------
@ldionne Do you think it's worthwhile to move this concept to a dedicated header?
================
Comment at: libcxx/include/string:1932-1933
+ void __init_with_size(_InputIterator __first, _Sentinel __last, size_type __sz) {
+ if (__libcpp_is_constant_evaluated())
+ __r_.first() = __rep();
+
----------------
ldionne wrote:
> Maybe call `__default_init()` here unconditionally for consistency with above?
Doing so breaks string tests.
================
Comment at: libcxx/test/std/ranges/range.utility/range.utility.conv/to_std_containers.pass.cpp:221
+ /*
+ types::for_each(container_adaptors<int>{}, []<class From>() {
+ types::for_each(container_adaptors<double>{}, []<class To>() {
----------------
ldionne wrote:
> Looks like a leftover!
Sorry about that!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142335/new/
https://reviews.llvm.org/D142335
More information about the libcxx-commits
mailing list