From 75034f7c0d218836840d4476d089a0a76f0dec7f Mon Sep 17 00:00:00 2001 From: Folkert van Heusden Date: Sat, 26 Oct 2024 19:06:24 +0200 Subject: [PATCH] code reduction --- iscsi-pdu.cpp | 32 -------------------------------- iscsi-pdu.h | 16 ++++++++-------- 2 files changed, 8 insertions(+), 40 deletions(-) diff --git a/iscsi-pdu.cpp b/iscsi-pdu.cpp index 34abb50..ba6d8c7 100644 --- a/iscsi-pdu.cpp +++ b/iscsi-pdu.cpp @@ -232,10 +232,6 @@ iscsi_pdu_login_request::iscsi_pdu_login_request(session *const ses): iscsi_pdu_ *login_req = { }; } -iscsi_pdu_login_request::~iscsi_pdu_login_request() -{ -} - bool has_CRC32C(const std::string & value) { auto parts = split(value, ","); @@ -400,10 +396,6 @@ iscsi_pdu_scsi_cmd::iscsi_pdu_scsi_cmd(session *const ses): iscsi_pdu_bhs(ses) assert(offsetof(__cdb_pdu_req__, CmdSN ) == 24); } -iscsi_pdu_scsi_cmd::~iscsi_pdu_scsi_cmd() -{ -} - bool iscsi_pdu_scsi_cmd::set(const uint8_t *const in, const size_t n) { if (iscsi_pdu_bhs::set(in, n) == false) { @@ -815,10 +807,6 @@ iscsi_pdu_nop_out::iscsi_pdu_nop_out(session *const ses): iscsi_pdu_bhs(ses) assert(sizeof(*nop_out) == 48); } -iscsi_pdu_nop_out::~iscsi_pdu_nop_out() -{ -} - std::optional iscsi_pdu_nop_out::get_response(scsi *const sd) { DOLOG(logging::ll_debug, "iscsi_pdu_nop_out::get_response", ses->get_endpoint_name(), "invoked"); @@ -841,10 +829,6 @@ iscsi_pdu_nop_in::iscsi_pdu_nop_in(session *const ses): iscsi_pdu_bhs(ses) assert(sizeof(*nop_in) == 48); } -iscsi_pdu_nop_in::~iscsi_pdu_nop_in() -{ -} - bool iscsi_pdu_nop_in::set(const iscsi_pdu_nop_out & reply_to) { auto ping_data = reply_to.get_data(); @@ -916,10 +900,6 @@ iscsi_pdu_text_request::iscsi_pdu_text_request(session *const ses): iscsi_pdu_bh *text_req = { }; } -iscsi_pdu_text_request::~iscsi_pdu_text_request() -{ -} - bool iscsi_pdu_text_request::set(const uint8_t *const in, const size_t n) { if (iscsi_pdu_bhs::set(in, n) == false) @@ -1020,10 +1000,6 @@ iscsi_pdu_logout_request::iscsi_pdu_logout_request(session *const ses): iscsi_pd *logout_req = { }; } -iscsi_pdu_logout_request::~iscsi_pdu_logout_request() -{ -} - bool iscsi_pdu_logout_request::set(const uint8_t *const in, const size_t n) { if (iscsi_pdu_bhs::set(in, n) == false) @@ -1091,10 +1067,6 @@ iscsi_pdu_taskman_request::iscsi_pdu_taskman_request(session *const ses): iscsi_ *taskman_req = { }; } -iscsi_pdu_taskman_request::~iscsi_pdu_taskman_request() -{ -} - bool iscsi_pdu_taskman_request::set(const uint8_t *const in, const size_t n) { if (iscsi_pdu_bhs::set(in, n) == false) @@ -1140,10 +1112,6 @@ iscsi_pdu_taskman_reply::iscsi_pdu_taskman_reply(session *const ses): iscsi_pdu_ assert(sizeof(*taskman_reply) == 48); } -iscsi_pdu_taskman_reply::~iscsi_pdu_taskman_reply() -{ -} - bool iscsi_pdu_taskman_reply::set(const iscsi_pdu_taskman_request & reply_to) { *taskman_reply = { }; diff --git a/iscsi-pdu.h b/iscsi-pdu.h index a614baf..8b68fd0 100644 --- a/iscsi-pdu.h +++ b/iscsi-pdu.h @@ -184,7 +184,7 @@ class iscsi_pdu_login_request : public iscsi_pdu_bhs // login request 0x03 public: iscsi_pdu_login_request(session *const ses); - virtual ~iscsi_pdu_login_request(); + virtual ~iscsi_pdu_login_request() { } std::vector get() const override; @@ -285,7 +285,7 @@ class iscsi_pdu_scsi_cmd : public iscsi_pdu_bhs // 0x01 public: iscsi_pdu_scsi_cmd(session *const ses); - virtual ~iscsi_pdu_scsi_cmd(); + virtual ~iscsi_pdu_scsi_cmd() { } bool set(const uint8_t *const in, const size_t n) override; std::vector get() const override; @@ -482,7 +482,7 @@ class iscsi_pdu_nop_out : public iscsi_pdu_bhs // NOP-Out 0x00 public: iscsi_pdu_nop_out(session *const ses); - virtual ~iscsi_pdu_nop_out(); + virtual ~iscsi_pdu_nop_out() { } const uint8_t *get_LUN() const { return nop_out->LUN; } uint32_t get_Itasktag() const { return nop_out->Itasktag; } @@ -520,7 +520,7 @@ class iscsi_pdu_nop_in : public iscsi_pdu_bhs // NOP-In public: iscsi_pdu_nop_in(session *const ses); - virtual ~iscsi_pdu_nop_in(); + virtual ~iscsi_pdu_nop_in() { } bool set(const iscsi_pdu_nop_out & reply_to); std::vector get() const override; @@ -600,7 +600,7 @@ class iscsi_pdu_text_request : public iscsi_pdu_bhs // text request 0x04 public: iscsi_pdu_text_request(session *const ses); - virtual ~iscsi_pdu_text_request(); + virtual ~iscsi_pdu_text_request() { } bool set(const uint8_t *const in, const size_t n) override; std::vector get() const override; @@ -681,7 +681,7 @@ class iscsi_pdu_logout_request : public iscsi_pdu_bhs // logout request 0x06 public: iscsi_pdu_logout_request(session *const ses); - virtual ~iscsi_pdu_logout_request(); + virtual ~iscsi_pdu_logout_request() { } bool set(const uint8_t *const in, const size_t n) override; std::vector get() const override; @@ -765,7 +765,7 @@ class iscsi_pdu_taskman_request : public iscsi_pdu_bhs // taskman request 0x02 public: iscsi_pdu_taskman_request(session *const ses); - virtual ~iscsi_pdu_taskman_request(); + virtual ~iscsi_pdu_taskman_request() { } bool set(const uint8_t *const in, const size_t n) override; std::vector get() const override; @@ -807,7 +807,7 @@ class iscsi_pdu_taskman_reply : public iscsi_pdu_bhs public: iscsi_pdu_taskman_reply(session *const ses); - virtual ~iscsi_pdu_taskman_reply(); + virtual ~iscsi_pdu_taskman_reply() { } bool set(const iscsi_pdu_taskman_request & reply_to); std::vector get() const override;