diff options
| author | Aleksander Morgado <aleksander@aleksander.es> | 2020-01-11 13:15:21 +0100 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2020-01-30 19:09:52 +0100 |
| commit | ac70712f1f5c12a2f85abf4e814efff8b41ff385 (patch) | |
| tree | 6cf30186ff3bd6518a06b0137f3ca46b0be06e9a | |
| parent | 7e67cbfe42815fdaae6d4852c5832542fd90904b (diff) | |
broadband-modem-mbim: fix warnings with -Wtype-limits
This change is exclusively to make the compiler happy, don't even like
it myself...
mm-broadband-modem-mbim.c: In function ‘signal_state_query_ready’:
/usr/include/glib-2.0/glib/gmacros.h:811:63: error: comparison of unsigned expression < 0 is always false [-Werror=type-limits]
811 | #define CLAMP(x, low, high) (((x) > (high)) ? (high) : (((x) < (low)) ? (low) : (x)))
| ^
mm-broadband-modem-mbim.c:1680:19: note: in expansion of macro ‘CLAMP’
1680 | quality = CLAMP (rssi == 99 ? 0 : rssi, 0, 31) * 100 / 31;
| ^~~~~
| -rw-r--r-- | src/mm-broadband-modem-mbim.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mm-broadband-modem-mbim.c b/src/mm-broadband-modem-mbim.c index 4910b299..f1f1b868 100644 --- a/src/mm-broadband-modem-mbim.c +++ b/src/mm-broadband-modem-mbim.c @@ -1677,7 +1677,7 @@ signal_state_query_ready (MbimDevice *device, guint32 quality; /* Normalize the quality. 99 means unknown, we default it to 0 */ - quality = CLAMP (rssi == 99 ? 0 : rssi, 0, 31) * 100 / 31; + quality = CLAMP (rssi == 99 ? 0 : (gint)rssi, 0, 31) * 100 / 31; g_task_return_int (task, quality); } else |