laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/33422 ) Change subject: gsm0502: cosmetic: use ARRAY_SIZE in gsm0502_fn2ccch_block() ...................................................................... gsm0502: cosmetic: use ARRAY_SIZE in gsm0502_fn2ccch_block() Change-Id: Ideb9907fcb5eff74508f44714e478ec46d435a5d --- M src/gsm/gsm0502.c 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/src/gsm/gsm0502.c b/src/gsm/gsm0502.c index e9e612a..e4a761d 100644 --- a/src/gsm/gsm0502.c +++ b/src/gsm/gsm0502.c @@ -293,5 +293,5 @@ */ int gsm0502_fn2ccch_block(uint32_t fn) { - return ccch_block_table[fn % 51]; + return ccch_block_table[fn % ARRAY_SIZE(ccch_block_table)]; } -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/33422 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Ideb9907fcb5eff74508f44714e478ec46d435a5d Gerrit-Change-Number: 33422 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge@osmocom.org> Gerrit-MessageType: merged