boostorg/iterator
1
0
Fork 1
mirror of https://github.com/boostorg/iterator.git synced 2025-07-30 04:47:20 +02:00
Code Issues Packages Projects Releases Wiki Activity

Use perfect forwarding in assignment to operator_brackets_proxy.

Browse Source 
Also apply noexcept markup to all of the proxy operators.
This commit is contained in:
Andrey Semashev
2025-06-09 03:37:01 +03:00
parent 7c77e52509
commit da00617e56
1 changed files with 12 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
include/boost/iterator/iterator_facade.hpp
View File

@ -9,6 +9,7 @@
#include <cstddef> #include <cstddef>
#include <memory> #include <memory>
#include <utility>
#include <type_traits> #include <type_traits>
#include <boost/config.hpp> #include <boost/config.hpp>
@ -346,21 +347,27 @@ class operator_brackets_proxy
// Iterator is actually an iterator_facade, so we do not have to // Iterator is actually an iterator_facade, so we do not have to
// go through iterator_traits to access the traits. // go through iterator_traits to access the traits.
using reference = typename Iterator::reference; using reference = typename Iterator::reference;
using value_type = typename Iterator::value_type;
public: public:
operator_brackets_proxy(Iterator const& iter) : explicit operator_brackets_proxy(Iterator const& iter) noexcept(std::is_nothrow_copy_constructible< Iterator >::value) :
m_iter(iter) m_iter(iter)
{} {}
operator reference() const operator reference() const noexcept(noexcept(*std::declval< Iterator const& >()))
{ {
return *m_iter; return *m_iter;
} }
operator_brackets_proxy& operator=(value_type const& val) template< typename T >
typename std::enable_if<
!std::is_same<
operator_brackets_proxy< Iterator >,
typename std::remove_cv< typename std::remove_reference< T >::type >::type
>::value,
operator_brackets_proxy&
>::type operator= (T&& val) noexcept(std::is_nothrow_assignable< reference, T&& >::value)
{ {
*m_iter = val; *m_iter = static_cast< T&& >(val);
return *this; return *this;
} }