From 33908dd86a09b417ef32145cd3b82927dd4540bd Mon Sep 17 00:00:00 2001 From: Lioncash Date: Tue, 3 Sep 2019 10:15:31 -0400 Subject: [PATCH] Listener: Simplify assignment within accept() We can just assign to the ret variable directly. --- lib/Listener.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/lib/Listener.cpp b/lib/Listener.cpp index ef96522..f24f459 100644 --- a/lib/Listener.cpp +++ b/lib/Listener.cpp @@ -91,13 +91,13 @@ void Listener::stop() { std::unique_ptr Listener::accept() { std::unique_lock lk{m_queueLock}; - if (m_endpointQueue.size()) { - std::unique_ptr ret; - ret = std::move(m_endpointQueue.front()); - m_endpointQueue.pop(); - return ret; + if (m_endpointQueue.empty()) { + return nullptr; } - return {}; + + auto ret = std::move(m_endpointQueue.front()); + m_endpointQueue.pop(); + return ret; } Listener::Listener() = default;