I noticed that std::string
's (really std::basic_string
's) move assignment operator is noexcept
. That makes sense to me. But then I noticed that none of the standard containers (e.g., std::vector
, std::deque
, std::list
, std::map
) declares its move assignment operator noexcept
. That makes less sense to me. A std::vector
, for example, is typically implemented as three pointers, and pointers can certainly be move-assigned without throwing an exception. Then I thought that maybe the problem is with moving the container's allocator, but std::string
's have allocators, too, so if that were the issue, I'd expect it to affect std::string
.
So why is std::string
's move assignment operator noexcept
, yet the move assignment operators for the standard containers are not?
I believe we're looking at a standards defect. The noexcept
specification, if it is to be applied to the move assignment operator, is somewhat complicated. And I believe this statement to be true whether we are talking about basic_string
or vector
.
Based on [container.requirements.general]/p7 my English translation of what a container move assignment operator is supposed to do is:
C& operator=(C&& c)
If
alloc_traits::propagate_on_container_move_assignment::value
istrue
, dumps resources, move assigns allocators, and transfers resources fromc
.If
alloc_traits::propagate_on_container_move_assignment::value
isfalse
andget_allocator() == c.get_allocator()
, dumps resources, and transfers resources fromc
.If
alloc_traits::propagate_on_container_move_assignment::value
isfalse
andget_allocator() != c.get_allocator()
, move assigns eachc[i]
.
Notes:
alloc_traits
refers to allocator_traits<allocator_type>
.
When alloc_traits::propagate_on_container_move_assignment::value
is true
the move assignment operator can be specified noexcept
because all it is going to is deallocate current resources and then pilfer resources from the source. Also in this case, the allocator must also be move assigned, and that move assignment must be noexcept
for the container's move assignment to be noexcept
.
When alloc_traits::propagate_on_container_move_assignment::value
is false
, and if the two allocators are equal, then it is going to do the same thing as #2. However one doesn't know if the allocators are equal until run time, so you can't base noexcept
on this possibility.
When alloc_traits::propagate_on_container_move_assignment::value
is false
, and if the two allocators are not equal, then one has to move assign each individual element. This may involve adding capacity or nodes to the target, and thus is intrinsically noexcept(false)
.
So in summary:
C& operator=(C&& c) noexcept( alloc_traits::propagate_on_container_move_assignment::value && is_nothrow_move_assignable<allocator_type>::value);
And I see no dependence on C::value_type
in the above spec and so I believe it should apply equally well to std::basic_string
despite C++11 specifying otherwise.
Update
In the comments below Columbo correctly points out that things have gradually been changing all the time. My comments above are relative to C++11.
For the draft C++17 (which seems stable at this point) things have changed somewhat:
If alloc_traits::propagate_on_container_move_assignment::value
is true
, the spec now requires the move assignment of the allocator_type
to not throw exceptions (17.6.3.5 [allocator.requirements]/p4). So one no longer needs to check is_nothrow_move_assignable<allocator_type>::value
.
alloc_traits::is_always_equal
has been added. If this is true, then one can determine at compile time that point 3 above can not throw because resources can be transferred.
So the new noexcept
spec for containers could be:
C& operator=(C&& c) noexcept( alloc_traits::propagate_on_container_move_assignment{} || alloc_traits::is_always_equal{});
And, for std::allocator<T>
, alloc_traits::propagate_on_container_move_assignment{}
and alloc_traits::is_always_equal{}
are both true.
Also now in the C++17 draft, both vector
and string
move assignment carry exactly this noexcept
specification. However the other containers carry variations of this noexcept
specification.
The safest thing to do if you care about this issue is to test explicit specializations of containers you care about. I've done exactly that for container<T>
for VS, libstdc++ and libc++ here:
http://howardhinnant.github.io/container_summary.html
This survey is about a year old, but as far as I know is still valid.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With