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)) {
-
dexter