fixeria has uploaded this change for review.
mobile: fix -Wmaybe-uninitialized in gsm48_rr_tx_meas_rep()
This is very unlikely to happen, because we set strongest to 127,
but anyway we don't want to see those warnings:
gsm48_rr.c: In function ‘gsm48_rr_tx_meas_rep.isra’:
gsm48_rr.c:3714:74: warning: ‘strongest_i’ may be used uninitialized [-Wmaybe-uninitialized]
3714 | if (rrmeas->nc_rxlev_dbm[i] == strongest && i <= strongest_i)
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~
gsm48_rr.c:3696:31: note: ‘strongest_i’ was declared here
3696 | int i, index, strongest_i;
| ^~~~~~~~~~~
Change-Id: I111438633ed48e16f7c43fc1b7a23ae6753a404d
---
M src/host/layer23/src/mobile/gsm48_rr.c
1 file changed, 21 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/13/37213/1
diff --git a/src/host/layer23/src/mobile/gsm48_rr.c b/src/host/layer23/src/mobile/gsm48_rr.c
index 83287c1..1750c57 100644
--- a/src/host/layer23/src/mobile/gsm48_rr.c
+++ b/src/host/layer23/src/mobile/gsm48_rr.c
@@ -3703,6 +3703,7 @@
/* get 6 strongest measurements */
strongest = 127; /* infinite */
+ strongest_i = -1; /* make gcc happy */
for (n = 0; n < 6; n++) {
current = -128; /* -infinite */
index = 0;
To view, visit change 37213. To unsubscribe, or for help writing mail filters, visit settings.