dexter has submitted this change. (
https://gerrit.osmocom.org/c/osmo-bts/+/32610 )
Change subject: l1sap: fix wording in comment
......................................................................
l1sap: fix wording in comment
The function rtppayload_is_valid() is called from the receiving RTP code
path. Lets use the word "forwarding" instead of "sending" to avoid the
impression something is sent (like sending RTP packets to the outside
world)
Change-Id: Ie7fcc53dea462b0d575b0c9ca73ba7507289eefe
---
M src/common/l1sap.c
1 file changed, 15 insertions(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, but someone else must approve
neels: Looks good to me, approved
diff --git a/src/common/l1sap.c b/src/common/l1sap.c
index 09e2d8c..a7eb830 100644
--- a/src/common/l1sap.c
+++ b/src/common/l1sap.c
@@ -1255,7 +1255,7 @@
if (resp_msg->len == 0)
return false;
- /* Avoid sending bw-efficient AMR to lower layers, most bts models
+ /* Avoid forwarding bw-efficient AMR to lower layers, most bts models
* don't support it. */
if (lchan->tch_mode == GSM48_CMODE_SPEECH_AMR &&
!rtppayload_is_octet_aligned(resp_msg->data, resp_msg->len)) {
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/32610
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ie7fcc53dea462b0d575b0c9ca73ba7507289eefe
Gerrit-Change-Number: 32610
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pma...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pma...@sysmocom.de>
Gerrit-Reviewer: neels <nhofm...@sysmocom.de>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged
