[libcxx-commits] [PATCH] D108047: [libcxx][ranges] Move `namespace views` into `namespace ranges` and add an alias.

Zoe Carver via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Aug 13 11:39:18 PDT 2021


zoecarver added inline comments.


================
Comment at: libcxx/test/std/ranges/range.adaptors/range.all/all.pass.cpp:88
   {
-    ASSERT_SAME_TYPE(decltype(std::views::all(View<true>())), View<true>);
-    static_assert(noexcept(std::views::all(View<true>())));
-    static_assert(!noexcept(std::views::all(View<false>())));
+    ASSERT_SAME_TYPE(decltype(std::ranges::views::all(View<true>())), View<true>);
+    static_assert(noexcept(std::ranges::views::all(View<true>())));
----------------
Intentionally testing this inconsistently so that we cover both `std::ranges::views` and `std::views`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108047



More information about the libcxx-commits mailing list