From c54bcf660f89a2d0a89e7df3bbe9a327a19dc39c Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Thu, 21 Jan 2021 20:44:59 +0800 Subject: [PATCH] crimson/net: move implementation of dtor into .cc file ceph_assert() is expanded into 5 lines of code. it'd help to speed up the compiling a little bit. Signed-off-by: Kefu Chai --- src/crimson/net/SocketMessenger.cc | 5 +++++ src/crimson/net/SocketMessenger.h | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/src/crimson/net/SocketMessenger.cc b/src/crimson/net/SocketMessenger.cc index db9421e79e2..0a86eb0736f 100644 --- a/src/crimson/net/SocketMessenger.cc +++ b/src/crimson/net/SocketMessenger.cc @@ -38,6 +38,11 @@ SocketMessenger::SocketMessenger(const entity_name_t& myname, nonce{nonce} {} +SocketMessenger::~SocketMessenger() +{ + ceph_assert(!listener); +} + seastar::future<> SocketMessenger::set_myaddrs(const entity_addrvec_t& addrs) { assert(seastar::this_shard_id() == master_sid); diff --git a/src/crimson/net/SocketMessenger.h b/src/crimson/net/SocketMessenger.h index 44c1d3c2137..9bd3cfc4f78 100644 --- a/src/crimson/net/SocketMessenger.h +++ b/src/crimson/net/SocketMessenger.h @@ -54,7 +54,7 @@ class SocketMessenger final : public Messenger { SocketMessenger(const entity_name_t& myname, const std::string& logic_name, uint32_t nonce); - ~SocketMessenger() override { ceph_assert(!listener); } + ~SocketMessenger() override; seastar::future<> set_myaddrs(const entity_addrvec_t& addr) override; -- 2.39.5