[libcxx-commits] [libcxx] [libc++] Add [[nodiscard]] to std::prev and std::next (PR #109550)
Louis Dionne via libcxx-commits
libcxx-commits at lists.llvm.org
Wed Sep 25 13:51:50 PDT 2024
================
@@ -43,25 +43,26 @@ next(_InputIter __x, typename iterator_traits<_InputIter>::difference_type __n =
namespace ranges {
struct __next {
template <input_or_output_iterator _Ip>
- _LIBCPP_HIDE_FROM_ABI constexpr _Ip operator()(_Ip __x) const {
+ [[nodiscard]] _LIBCPP_HIDE_FROM_ABI constexpr _Ip operator()(_Ip __x) const {
++__x;
return __x;
----------------
ldionne wrote:
Not attached to this line: Can you please add tests for this? It should be sufficient to add entries for `std::next`, `std::prev` and their ranges counterparts in `libcxx/test/libcxx/diagnostics/iterator.nodiscard.verify.cpp`.
https://github.com/llvm/llvm-project/pull/109550
More information about the libcxx-commits
mailing list