Enable Build with GCC 9.2 62/221962/1 accepted/tizen/unified/20200115.123330 submit/tizen/20200108.121304 submit/tizen/20200110.034022 submit/tizen/20200113.004849 submit/tizen/20200113.072030 submit/tizen/20200113.114913 submit/tizen/20200114.035328 submit/tizen/20200114.073857 submit/tizen/20200114.120640 submit/tizen/20200115.100804
authorAbhimanyu Swami <abhimanyu1.s@samsung.com>
Wed, 8 Jan 2020 11:44:12 +0000 (17:14 +0530)
committerAbhimanyu Swami <abhimanyu1.s@samsung.com>
Wed, 8 Jan 2020 11:44:12 +0000 (17:14 +0530)
Change-Id: I783b315f0208221b668f6168a23257673a396966
Signed-off-by: Abhimanyu Swami <abhimanyu1.s@samsung.com>
src/api/maps_address.cpp
src/api/maps_area.cpp
src/api/maps_coordinates.cpp
src/maps_util.h

index 47e7938..400bcfe 100644 (file)
@@ -528,7 +528,7 @@ EXPORT_API int maps_address_list_destroy(maps_address_list_h address_list)
 bool maps_address_is_valid(const maps_address_h address)
 {
        if (!maps_condition_check_maps_feature())
-               return MAPS_ERROR_NOT_SUPPORTED;
+               return false;
        if (!address) return false;
 
        bool ret = false;
index 01967d0..98df919 100644 (file)
@@ -135,7 +135,7 @@ EXPORT_API int maps_area_clone(const maps_area_h origin, maps_area_h *cloned)
 bool maps_area_is_valid(const maps_area_h area)
 {
        if (!maps_condition_check_maps_feature())
-               return MAPS_ERROR_NOT_SUPPORTED;
+               return false;
        if (!area) return false;
 
        bool ret = true;
index 18d147e..c5a8bf5 100644 (file)
@@ -280,15 +280,26 @@ EXPORT_API int maps_coordinates_list_foreach(maps_coordinates_list_h coordinates
 
 bool maps_coordinates_is_valid(const maps_coordinates_h coordinates)
 {
-       if (!coordinates)
-               return false;
+       bool ret_val = true;
 
-       maps_coordinates_s *coord = (maps_coordinates_s *)coordinates;
+        if (!coordinates)
+                return false;
 
-       MAPS_CHECK_CONDITION(coord->latitude >= -90 && coord->latitude <= 90, MAPS_ERROR_INVALID_PARAMETER,
-               "MAPS_ERROR_INVALID_PARAMETER");
-       MAPS_CHECK_CONDITION(coord->longitude >= -180 && coord->longitude <= 180, MAPS_ERROR_INVALID_PARAMETER,
-               "MAPS_ERROR_INVALID_PARAMETER");
+        maps_coordinates_s *coord = (maps_coordinates_s *)coordinates;
+
+        if (!(coord->latitude >= -90 && coord->latitude <= 90)) {
+                MAPS_LOGE("%s(0x%08x)", "MAPS_ERROR_INVALID_PARAMETER", MAPS_ERROR_INVALID_PARAMETER);
+                ret_val = false;
+                goto END;
+        }
+
+        if (!(coord->longitude >= -180 && coord->longitude <= 180)) {
+                MAPS_LOGE("%s(0x%08x)", "MAPS_ERROR_INVALID_PARAMETER", MAPS_ERROR_INVALID_PARAMETER);
+                ret_val = false;
+                goto END;
+        }
+
+END:
+        return ret_val;
 
-       return true;
 }
index f156406..ee1f636 100755 (executable)
@@ -54,7 +54,7 @@
                if (condition) { \
                } else { \
                        MAPS_LOGE("%s(0x%08x)", msg, error); \
-                       return error; \
+                       return error;\
                } \
        } while (0)