pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-netif/+/30694 ) Change subject: amr: osmo_amr_bwe_to_oa() define variable as unsigned ...................................................................... amr: osmo_amr_bwe_to_oa() define variable as unsigned There's no need to have the variable as signed anymore since the loop was modified in the previous patch. osmo_amr_bytes() returns a size_t (unsigned). Change-Id: I8aa6b5f6d3334e152a62b7c28aac3f881f027894 --- M src/amr.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-netif refs/changes/94/30694/1 diff --git a/src/amr.c b/src/amr.c index 8cd856d..a40e496 100644 --- a/src/amr.c +++ b/src/amr.c @@ -194,7 +194,7 @@ /* The header is only valid after shifting first two bytes to OA mode */ struct amr_hdr *oa_hdr; unsigned int i; - int oa_payload_len; + unsigned int oa_payload_len; memset(buf, 0, sizeof(buf)); -- To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/30694 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-netif Gerrit-Branch: master Gerrit-Change-Id: I8aa6b5f6d3334e152a62b7c28aac3f881f027894 Gerrit-Change-Number: 30694 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange