From c8e4c2e751e66d0e0c86ea4a2055331e9eedca88 Mon Sep 17 00:00:00 2001 From: Aleksander Morgado Date: Sat, 11 Jan 2020 13:23:24 +0100 Subject: [PATCH] broadband-modem-mbim: fix warnings with -Wtype-limits MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit mm-broadband-modem-mbim.c: In function ‘basic_connect_notification_signal_state’: /usr/include/glib-2.0/glib/gmacros.h:811:63: error: comparison of unsigned expression < 0 is always false [-Werror=type-limits] 811 | ine CLAMP(x, low, high) (((x) > (high)) ? (high) : (((x) < (low)) ? (low) : (x))) | ^ mm-broadband-modem-mbim.c:2918:19: note: in expansion of macro ‘CLAMP’ 2918 | quality = CLAMP (rssi == 99 ? 0 : rssi, 0, 31) * 100 / 31; | ^~~~~ --- src/mm-broadband-modem-mbim.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mm-broadband-modem-mbim.c b/src/mm-broadband-modem-mbim.c index f1f1b868..4c146704 100644 --- a/src/mm-broadband-modem-mbim.c +++ b/src/mm-broadband-modem-mbim.c @@ -2920,7 +2920,7 @@ basic_connect_notification_signal_state (MMBroadbandModemMbim *self, 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; mm_dbg ("Signal state indication: %u --> %u%%", rssi, quality); mm_iface_modem_update_signal_quality (MM_IFACE_MODEM (self), quality);