fixeria has submitted this change. (
https://gerrit.osmocom.org/c/libosmo-gprs/+/31948 )
Change subject: rlcmac: cosmetic: use ARRAY_SIZE() in handle_pkt_dl_ass()
......................................................................
rlcmac: cosmetic: use ARRAY_SIZE() in handle_pkt_dl_ass()
Change-Id: I98cff9c38ef8d44669ae233f2cf1c7d815e27ff8
---
M src/rlcmac/tbf_dl_ass_fsm.c
1 file changed, 10 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, approved
diff --git a/src/rlcmac/tbf_dl_ass_fsm.c b/src/rlcmac/tbf_dl_ass_fsm.c
index 9f13e75..74e1853 100644
--- a/src/rlcmac/tbf_dl_ass_fsm.c
+++ b/src/rlcmac/tbf_dl_ass_fsm.c
@@ -98,7 +98,6 @@
{
const Packet_Downlink_Assignment_t *dlass =
&d->dl_block->u.Packet_Downlink_Assignment;
- unsigned int i;
if (dlass->Exist_DOWNLINK_TFI_ASSIGNMENT)
ctx->alloc.dl_tfi = dlass->DOWNLINK_TFI_ASSIGNMENT;
@@ -110,7 +109,7 @@
ctx->tbf_starting_time =
TBF_Starting_Frame_Number_to_fn(&dlass->TBF_Starting_Time, d->fn);
ctx->alloc.num_ts = 0;
- for (i = 0; i < 8; i++) {
+ for (unsigned int i = 0; i < ARRAY_SIZE(ctx->alloc.ts); i++) {
ctx->alloc.ts[i].allocated = (dlass->TIMESLOT_ALLOCATION >> i)
& 0x01;
if (ctx->alloc.ts[i].allocated)
ctx->alloc.num_ts++;
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/31948
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I98cff9c38ef8d44669ae233f2cf1c7d815e27ff8
Gerrit-Change-Number: 31948
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged
