Change in ...libosmocore[master]: codec/ecu_fr: Mark input TCH frame as 'const' as we only read it

laforge gerrit-no-reply at lists.osmocom.org
Thu Aug 1 19:08:06 UTC 2019


laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmocore/+/15028


Change subject: codec/ecu_fr: Mark input TCH frame as 'const' as we only read it
......................................................................

codec/ecu_fr: Mark input TCH frame as 'const' as we only read it

Change-Id: I64c6d3dc08ff87b673ba6225f98546e86f91bcfd
---
M include/osmocom/codec/ecu.h
M src/codec/ecu_fr.c
2 files changed, 2 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/28/15028/1

diff --git a/include/osmocom/codec/ecu.h b/include/osmocom/codec/ecu.h
index f7a29a0..ec0a2f8 100644
--- a/include/osmocom/codec/ecu.h
+++ b/include/osmocom/codec/ecu.h
@@ -11,5 +11,5 @@
 	uint8_t frame_backup[GSM_FR_BYTES];
 };
 
-void osmo_ecu_fr_reset(struct osmo_ecu_fr_state *state, uint8_t *frame);
+void osmo_ecu_fr_reset(struct osmo_ecu_fr_state *state, const uint8_t *frame);
 int osmo_ecu_fr_conceal(struct osmo_ecu_fr_state *state, uint8_t *frame);
diff --git a/src/codec/ecu_fr.c b/src/codec/ecu_fr.c
index 20a0f89..ef42ea9 100644
--- a/src/codec/ecu_fr.c
+++ b/src/codec/ecu_fr.c
@@ -130,7 +130,7 @@
  * \param[in] state The state object for the ECU
  * \param[out] frame The valid frame (GSM_FR_BYTES bytes in RTP payload format)
  */
-void osmo_ecu_fr_reset(struct osmo_ecu_fr_state *state, uint8_t *frame)
+void osmo_ecu_fr_reset(struct osmo_ecu_fr_state *state, const uint8_t *frame)
 {
 	state->subsequent_lost_frame = false;
 	memcpy(state->frame_backup, frame, GSM_FR_BYTES);

-- 
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/15028
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I64c6d3dc08ff87b673ba6225f98546e86f91bcfd
Gerrit-Change-Number: 15028
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge at gnumonks.org>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190801/e97712c2/attachment.html>


More information about the gerrit-log mailing list