Harald Welte has submitted this change and it was merged. (
https://gerrit.osmocom.org/10775 )
Change subject: codec_pref.c: Add comment in expected switch case without break
......................................................................
codec_pref.c: Add comment in expected switch case without break
Change-Id: I568353f070f049003326306106dce89b35deb92f
---
M src/osmo-bsc/codec_pref.c
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Harald Welte: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/osmo-bsc/codec_pref.c b/src/osmo-bsc/codec_pref.c
index cbe8ef4..2a9be6e 100644
--- a/src/osmo-bsc/codec_pref.c
+++ b/src/osmo-bsc/codec_pref.c
@@ -44,6 +44,7 @@
break;
case GSM0808_PERM_HR2:
LOGP(DMSC, LOGL_FATAL, "Speech HR2 was never specified!\n");
+ /* fall through */
default:
LOGP(DMSC, LOGL_FATAL, "Unsupported permitted speech %s
selected, "
"assuming AMR as channel mode...\n",
--
To view, visit https://gerrit.osmocom.org/10775
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I568353f070f049003326306106dce89b35deb92f
Gerrit-Change-Number: 10775
Gerrit-PatchSet: 2
Gerrit-Owner: Pau Espin Pedrol <pes...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)
