Commit d062f7bc authored by Piotr Tworek's avatar Piotr Tworek Committed by Commit Bot

Fix intrusive_heap.h build when exceptions are enabled.

This not visible in default chromium setup since it disables exceptions
by default. The code does however fail to compile if one enables them:

../../base/containers/intrusive_heap.h:717:48: error: 'operator=' is missing exception specification 'noexcept'
IntrusiveHeap<T, Compare, HeapHandleAccessor>::operator=(
                                               ^
../../base/containers/intrusive_heap.h:290:18: note: previous declaration is here
  IntrusiveHeap& operator=(IntrusiveHeap&&) noexcept;

Change-Id: I8c633333ae17002ba0b643e6c88e463d28de0086
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1815862Reviewed-by: default avatarDaniel Cheng <dcheng@chromium.org>
Commit-Queue: Piotr Tworek <ptworek@vewd.com>
Cr-Commit-Position: refs/heads/master@{#699661}
parent 1ac1dc17
...@@ -716,7 +716,7 @@ IntrusiveHeap<T, Compare, HeapHandleAccessor>::~IntrusiveHeap() { ...@@ -716,7 +716,7 @@ IntrusiveHeap<T, Compare, HeapHandleAccessor>::~IntrusiveHeap() {
template <typename T, typename Compare, typename HeapHandleAccessor> template <typename T, typename Compare, typename HeapHandleAccessor>
IntrusiveHeap<T, Compare, HeapHandleAccessor>& IntrusiveHeap<T, Compare, HeapHandleAccessor>&
IntrusiveHeap<T, Compare, HeapHandleAccessor>::operator=( IntrusiveHeap<T, Compare, HeapHandleAccessor>::operator=(
IntrusiveHeap&& other) { IntrusiveHeap&& other) noexcept {
clear(); clear();
impl_ = std::move(other.impl_); impl_ = std::move(other.impl_);
return *this; return *this;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment