Skip to content

Commit 0339a8f

Browse files
pdcp: methods of final classes don't need to be declared as final
1 parent 86bb33f commit 0339a8f

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

lib/pdcp/pdcp_entity_rx.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ class pdcp_entity_rx final : public pdcp_entity_tx_rx_base,
9191
/*
9292
* Security configuration
9393
*/
94-
void configure_security(security::sec_128_as_config sec_cfg_) final
94+
void configure_security(security::sec_128_as_config sec_cfg_) override
9595
{
9696
srsran_assert((is_srb() && sec_cfg_.domain == security::sec_domain::rrc) ||
9797
(is_drb() && sec_cfg_.domain == security::sec_domain::up),
@@ -121,7 +121,7 @@ class pdcp_entity_rx final : public pdcp_entity_tx_rx_base,
121121
logger.log_info("128 K_enc: {}", sec_cfg.k_128_enc);
122122
}
123123

124-
void set_integrity_protection(security::integrity_enabled integrity_enabled_) final
124+
void set_integrity_protection(security::integrity_enabled integrity_enabled_) override
125125
{
126126
if (integrity_enabled_ == security::integrity_enabled::on) {
127127
if (!sec_cfg.k_128_int.has_value()) {
@@ -136,7 +136,7 @@ class pdcp_entity_rx final : public pdcp_entity_tx_rx_base,
136136
integrity_enabled = integrity_enabled_;
137137
logger.log_info("Set integrity_enabled={}", integrity_enabled);
138138
}
139-
void set_ciphering(security::ciphering_enabled ciphering_enabled_) final
139+
void set_ciphering(security::ciphering_enabled ciphering_enabled_) override
140140
{
141141
ciphering_enabled = ciphering_enabled_;
142142
logger.log_info("Set ciphering_enabled={}", ciphering_enabled);

lib/pdcp/pdcp_entity_tx.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ class pdcp_entity_tx final : public pdcp_entity_tx_rx_base,
109109
/*
110110
* SDU/PDU handlers
111111
*/
112-
void handle_sdu(byte_buffer sdu) final;
112+
void handle_sdu(byte_buffer sdu) override;
113113

114114
void handle_transmit_notification(uint32_t notif_sn) override;
115115
void handle_delivery_notification(uint32_t notif_sn) override;
@@ -124,7 +124,7 @@ class pdcp_entity_tx final : public pdcp_entity_tx_rx_base,
124124
void handle_status_report(byte_buffer_chain status);
125125

126126
// Status handler interface
127-
void on_status_report(byte_buffer_chain status) final { handle_status_report(std::move(status)); }
127+
void on_status_report(byte_buffer_chain status) override { handle_status_report(std::move(status)); }
128128

129129
/*
130130
* Header helpers
@@ -153,7 +153,7 @@ class pdcp_entity_tx final : public pdcp_entity_tx_rx_base,
153153
*/
154154
void configure_security(security::sec_128_as_config sec_cfg,
155155
security::integrity_enabled integrity_enabled_,
156-
security::ciphering_enabled ciphering_enabled_) final;
156+
security::ciphering_enabled ciphering_enabled_) override;
157157

158158
/// Sends a status report, as specified in TS 38.323, Sec. 5.4.
159159
void send_status_report();

0 commit comments

Comments
 (0)