laforge submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
gsm0502: cosmetic: use ARRAY_SIZE in gsm0502_fn2ccch_block()

Change-Id: Ideb9907fcb5eff74508f44714e478ec46d435a5d
---
M src/gsm/gsm0502.c
1 file changed, 10 insertions(+), 1 deletion(-)

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