From: Kefu Chai <kchai@redhat.com>
Date: Thu, 21 Jan 2021 12:44:59 +0000 (+0800)
Subject: crimson/net: move implementation of dtor into .cc file
X-Git-Tag: v17.0.0~4^2~5
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=c54bcf660f89a2d0a89e7df3bbe9a327a19dc39c;p=ceph.git

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 <kchai@redhat.com>
---

diff --git a/src/crimson/net/SocketMessenger.cc b/src/crimson/net/SocketMessenger.cc
index db9421e79e28..0a86eb0736f5 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 44c1d3c21375..9bd3cfc4f784 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;