[PATCH v2] broadband-modem-qmi: fix incorrect conversions from double to gint32

Ben Chan benchan at chromium.org
Wed Jul 10 23:31:15 PDT 2013


This patch fixes the following incorrect conversions from double to gint32:

mm-broadband-modem-qmi.c:4785:27: error: implicit conversion from 'double' to 'gint32' (aka 'int')
changes value from 2.225073858507201E-308 to 0 [-Werror,-Wliteral-conversion]
    gint32 bs_longitude = MM_LOCATION_LONGITUDE_UNKNOWN;
           ~~~~~~~~~~~~   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~

mm-broadband-modem-qmi.c:4786:26: error: implicit conversion from 'double' to 'gint32' (aka 'int')
changes value from 2.225073858507201E-308 to 0 [-Werror,-Wliteral-conversion]
    gint32 bs_latitude = MM_LOCATION_LATITUDE_UNKNOWN;
           ~~~~~~~~~~~   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
---
 src/mm-broadband-modem-qmi.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/mm-broadband-modem-qmi.c b/src/mm-broadband-modem-qmi.c
index efaefb3..0c96b1a 100644
--- a/src/mm-broadband-modem-qmi.c
+++ b/src/mm-broadband-modem-qmi.c
@@ -4782,8 +4782,8 @@ common_process_serving_system_cdma (MMBroadbandModemQmi *self,
     guint16 sid = 0;
     guint16 nid = 0;
     guint16 bs_id = 0;
-    gint32 bs_longitude = MM_LOCATION_LONGITUDE_UNKNOWN;
-    gint32 bs_latitude = MM_LOCATION_LATITUDE_UNKNOWN;
+    gint32 bs_longitude = G_MININT32;
+    gint32 bs_latitude = G_MININT32;
 
     if (response_output)
         qmi_message_nas_get_serving_system_output_get_serving_system (
@@ -4933,11 +4933,11 @@ common_process_serving_system_cdma (MMBroadbandModemQmi *self,
     /* Longitude and latitude given in units of 0.25 secs
      * Note that multiplying by 0.25 is like dividing by 4, so 60*60*4=14400 */
 #define QMI_LONGITUDE_TO_DEGREES(longitude)       \
-    (longitude != MM_LOCATION_LONGITUDE_UNKNOWN ? \
+    (longitude != G_MININT32 ? \
      (((gdouble)longitude) / 14400.0) :           \
      MM_LOCATION_LONGITUDE_UNKNOWN)
 #define QMI_LATITUDE_TO_DEGREES(latitude)         \
-    (latitude != MM_LOCATION_LATITUDE_UNKNOWN ?   \
+    (latitude != G_MININT32 ?   \
      (((gdouble)latitude) / 14400.0) :            \
      MM_LOCATION_LATITUDE_UNKNOWN)
 
-- 
1.8.3



More information about the ModemManager-devel mailing list