pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved
coding: Use ARRAY_SIZE macro

Change-Id: I6b7a2a3a7be6a1d92038ff3b249e539fcd0f639e
---
M src/coding/gsm0503_coding.c
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/coding/gsm0503_coding.c b/src/coding/gsm0503_coding.c
index 493f957..3ad2861 100644
--- a/src/coding/gsm0503_coding.c
+++ b/src/coding/gsm0503_coding.c
@@ -547,7 +547,7 @@
return res;

coded_len = osmo_conv_encode(code, output, recoded);
- OSMO_ASSERT(sizeof(recoded) / sizeof(recoded[0]) >= coded_len);
+ OSMO_ASSERT(ARRAY_SIZE(recoded) >= coded_len);

/* Count bit errors */
if (n_errors) {

To view, visit change 28104. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I6b7a2a3a7be6a1d92038ff3b249e539fcd0f639e
Gerrit-Change-Number: 28104
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged