dexter submitted this change.

View Change


Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve neels: Looks good to me, approved
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(-)

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 change 32610. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ie7fcc53dea462b0d575b0c9ca73ba7507289eefe
Gerrit-Change-Number: 32610
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged