Commit 7d5ef160 authored by Jose Dapena Paz's avatar Jose Dapena Paz Committed by Commit Bot

mojo: add missing noexcept to move operator/constructor in PendingRemoteState

PendingRemote has moved to declare its move operator/constructors noexcept. But
it requires its members to do that too. We were missing noexcept in
PendingRemoteState.

Bug: 819294
Change-Id: I95af4c8d96a89c5a7991f414624a2a47d16494e6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1713527Reviewed-by: default avatarKen Rockot <rockot@google.com>
Commit-Queue: José Dapena Paz <jose.dapena@lge.com>
Cr-Commit-Position: refs/heads/master@{#679938}
parent c9400d06
...@@ -13,11 +13,12 @@ PendingRemoteState::PendingRemoteState(ScopedMessagePipeHandle pipe, ...@@ -13,11 +13,12 @@ PendingRemoteState::PendingRemoteState(ScopedMessagePipeHandle pipe,
uint32_t version) uint32_t version)
: pipe(std::move(pipe)), version(version) {} : pipe(std::move(pipe)), version(version) {}
PendingRemoteState::PendingRemoteState(PendingRemoteState&&) = default; PendingRemoteState::PendingRemoteState(PendingRemoteState&&) noexcept = default;
PendingRemoteState::~PendingRemoteState() = default; PendingRemoteState::~PendingRemoteState() = default;
PendingRemoteState& PendingRemoteState::operator=(PendingRemoteState&& other) { PendingRemoteState& PendingRemoteState::operator=(
PendingRemoteState&& other) noexcept {
reset(); reset();
pipe = std::move(other.pipe); pipe = std::move(other.pipe);
version = other.version; version = other.version;
......
...@@ -21,11 +21,11 @@ struct COMPONENT_EXPORT(MOJO_CPP_BINDINGS_BASE) PendingRemoteState { ...@@ -21,11 +21,11 @@ struct COMPONENT_EXPORT(MOJO_CPP_BINDINGS_BASE) PendingRemoteState {
PendingRemoteState(); PendingRemoteState();
PendingRemoteState(ScopedMessagePipeHandle pipe, uint32_t version); PendingRemoteState(ScopedMessagePipeHandle pipe, uint32_t version);
PendingRemoteState(const PendingRemoteState&) = delete; PendingRemoteState(const PendingRemoteState&) = delete;
PendingRemoteState(PendingRemoteState&&); PendingRemoteState(PendingRemoteState&&) noexcept;
~PendingRemoteState(); ~PendingRemoteState();
PendingRemoteState& operator=(const PendingRemoteState&) = delete; PendingRemoteState& operator=(const PendingRemoteState&) = delete;
PendingRemoteState& operator=(PendingRemoteState&&); PendingRemoteState& operator=(PendingRemoteState&&) noexcept;
void reset(); void reset();
......
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