[libcxx-commits] [PATCH] D145981: [libc++] Assert that lengths fit in difference_type

[David Benjamin via Phabricator via libcxx-commits]

davidben added inline comments.


================
Comment at: libcxx/test/libcxx/strings/string.view/assert.ctor.length.pass.cpp:10-12
+// UNSUPPORTED: c++11, c++14
+
+// UNSUPPORTED: c++03
----------------
Mordante wrote:
> Why is this not supported in these language versions? We have enabled `string_view` in older language versions.
I just copied that from `assert.ctor.pointer.pass.cpp`. :-) Removed.


================
Comment at: libcxx/test/libcxx/strings/string.view/assert.ctor.length.pass.cpp:15
+// ADDITIONAL_COMPILE_FLAGS: -D_LIBCPP_ENABLE_ASSERTIONS=1
+
+#include <string_view>
----------------
Mordante wrote:
> Please add the function's synopsis here, best look for other tests for examples.
Something like this?


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

https://reviews.llvm.org/D145981