[libcxx-commits] [PATCH] D131681: [libc++] Add a missing assertion in std::span's constructor

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Aug 11 07:29:43 PDT 2022


ldionne created this revision.
Herald added a project: All.
ldionne requested review of this revision.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

Also, add missing tests for assertions in span constructors. Now I
believe that all of std::span's API should be hardened, and all the
assertions should have a corresponding test.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D131681

Files:
  libcxx/include/span
  libcxx/test/libcxx/containers/views/views.span/span.cons/assert.iter_sent.pass.cpp
  libcxx/test/libcxx/containers/views/views.span/span.cons/assert.iter_size.pass.cpp
  libcxx/test/libcxx/containers/views/views.span/span.cons/assert.other_span.pass.cpp
  libcxx/test/libcxx/containers/views/views.span/span.cons/assert.range.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D131681.451855.patch
Type: text/x-patch
Size: 7511 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220811/bc67d4e9/attachment.bin>


More information about the libcxx-commits mailing list