From ece7f3ffd216e307f74a4ca3ca80011252ee4be7 Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Sun, 14 Apr 2019 23:31:24 +0800 Subject: [PATCH] messages: do not use MessageSubType anymore so we can drop it in future commit Signed-off-by: Kefu Chai --- src/messages/MMonQuorumService.h | 9 ++++----- src/messages/MOSDFastDispatchOp.h | 9 ++++----- src/messages/MOSDPeeringOp.h | 7 +++---- src/messages/PaxosServiceMessage.h | 7 +++---- 4 files changed, 14 insertions(+), 18 deletions(-) diff --git a/src/messages/MMonQuorumService.h b/src/messages/MMonQuorumService.h index 7ad1eeb50c83b..a2cf527b5ecd4 100644 --- a/src/messages/MMonQuorumService.h +++ b/src/messages/MMonQuorumService.h @@ -16,19 +16,18 @@ #include "msg/Message.h" -class MMonQuorumService : public MessageSubType { +class MMonQuorumService : public Message { public: epoch_t epoch = 0; version_t round = 0; protected: -template - MMonQuorumService(Args&&... args) : MessageSubType(std::forward(args)...) {} - + MMonQuorumService(int type, int head) + : Message{type, head, 1} + {} ~MMonQuorumService() override { } public: - void set_epoch(epoch_t e) { epoch = e; } diff --git a/src/messages/MOSDFastDispatchOp.h b/src/messages/MOSDFastDispatchOp.h index 1f5f7abe22305..d1103fb8810f7 100644 --- a/src/messages/MOSDFastDispatchOp.h +++ b/src/messages/MOSDFastDispatchOp.h @@ -7,12 +7,11 @@ #include "msg/Message.h" #include "osd/osd_types.h" -class MOSDFastDispatchOp : public MessageSubType { +class MOSDFastDispatchOp : public Message { public: - -template - MOSDFastDispatchOp(Args&&... args) : MessageSubType(std::forward(args)...) {} - + MOSDFastDispatchOp(int t, int version, int compat_version) + : Message{t, version, compat_version} + {} virtual epoch_t get_map_epoch() const = 0; virtual epoch_t get_min_epoch() const { return get_map_epoch(); diff --git a/src/messages/MOSDPeeringOp.h b/src/messages/MOSDPeeringOp.h index b6ea078eac38d..f2fdcf12936e2 100644 --- a/src/messages/MOSDPeeringOp.h +++ b/src/messages/MOSDPeeringOp.h @@ -7,11 +7,10 @@ #include "osd/osd_types.h" #include "osd/PGPeeringEvent.h" -class MOSDPeeringOp : public MessageSubType { +class MOSDPeeringOp : public Message { public: - -template - MOSDPeeringOp(Args&&... args) : MessageSubType(std::forward(args)...) {} + MOSDPeeringOp(int t, int version, int compat_version) + : Message{t, version, compat_version} {} void print(std::ostream& out) const override final { out << get_type_name() << "(" diff --git a/src/messages/PaxosServiceMessage.h b/src/messages/PaxosServiceMessage.h index 1cdb7a3c01b0e..8bdad564dfa5a 100644 --- a/src/messages/PaxosServiceMessage.h +++ b/src/messages/PaxosServiceMessage.h @@ -4,10 +4,9 @@ #define CEPH_PAXOSSERVICEMESSAGE_H #include "msg/Message.h" -#include "msg/MessageRef.h" #include "mon/Session.h" -class PaxosServiceMessage : public MessageSubType { +class PaxosServiceMessage : public Message { public: version_t version; __s16 deprecated_session_mon; @@ -18,11 +17,11 @@ public: epoch_t rx_election_epoch; PaxosServiceMessage() - : MessageSubType(MSG_PAXOS), + : Message{MSG_PAXOS}, version(0), deprecated_session_mon(-1), deprecated_session_mon_tid(0), rx_election_epoch(0) { } PaxosServiceMessage(int type, version_t v, int enc_version=1, int compat_enc_version=0) - : MessageSubType(type, enc_version, compat_enc_version), + : Message{type, enc_version, compat_enc_version}, version(v), deprecated_session_mon(-1), deprecated_session_mon_tid(0), rx_election_epoch(0) { } protected: -- 2.39.5