[MERGED] osmo-msc[master]: a_iface: Consistent and understandable function names

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Mon Feb 12 11:23:37 UTC 2018


Harald Welte has submitted this change and it was merged.

Change subject: a_iface: Consistent and understandable function names
......................................................................


a_iface: Consistent and understandable function names

Change-Id: I36894e68e4b1f25f266e941c39a44e0171d24689
---
M src/libmsc/a_iface.c
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/libmsc/a_iface.c b/src/libmsc/a_iface.c
index ed6dcdb..86c2e04 100644
--- a/src/libmsc/a_iface.c
+++ b/src/libmsc/a_iface.c
@@ -234,7 +234,7 @@
 }
 
 /* Convert speech version field */
-static uint8_t convert_l3_sv_to_A_sv(int speech_ver)
+static uint8_t convert_speech_version_l3_to_A(int speech_ver)
 {
 	/* The speech versions that are transmitted in the Bearer capability
 	 * information element, that is transmitted on the Layer 3 (CC)
@@ -270,7 +270,7 @@
 }
 
 /* Convert speech preference field */
-static uint8_t convert_l3_prev_to_A_pref(int radio)
+static uint8_t convert_speech_pref_l3_to_A(int radio)
 {
 	/* The Radio channel requirement field that is transmitted in the
 	 * Bearer capability information element, that is transmitted on the
@@ -309,7 +309,7 @@
 	for (i = 0; i < ARRAY_SIZE(bc->speech_ver); i++) {
 		if (bc->speech_ver[i] == -1)
 			break;
-		sv = convert_l3_sv_to_A_sv(bc->speech_ver[i]);
+		sv = convert_speech_version_l3_to_A(bc->speech_ver[i]);
 		if (sv != 0xFF) {
 			/* Detect if something else than
 			 * GSM HR V1 is supported */
@@ -331,7 +331,7 @@
 		 * preference and assume a preference for full rate. */
 		ct->ch_rate_type = GSM0808_SPEECH_FULL_BM;
 	else
-		ct->ch_rate_type = convert_l3_prev_to_A_pref(bc->radio);
+		ct->ch_rate_type = convert_speech_pref_l3_to_A(bc->radio);
 
 	if (count)
 		return 0;

-- 
To view, visit https://gerrit.osmocom.org/6379
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I36894e68e4b1f25f266e941c39a44e0171d24689
Gerrit-PatchSet: 1
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list