[libcxx-commits] [libcxx] [libc++][iterator] Applied `[[nodiscard]]` (PR #172200)

[Nikolas Klauser via libcxx-commits]

================
@@ -80,9 +80,9 @@ class __iterator_with_data {
     return __tmp;
   }
 
-  constexpr _LIBCPP_HIDE_FROM_ABI iter_reference_t<_Iterator> operator*() const { return *__iter_; }
+  [[nodiscard]] constexpr _LIBCPP_HIDE_FROM_ABI iter_reference_t<_Iterator> operator*() const { return *__iter_; }
----------------
philnik777 wrote:

Yeah, let's separate library-internal stuff from changes to the public interface.

https://github.com/llvm/llvm-project/pull/172200