Merge pull request #9692 from naveenkaje/fix_ublox_celluar_warning

Cruz Monrreal · web-flow · commit e310458a88b5 · 2019-02-13T17:12:51.000-06:00
target: UBLOX: Fix build warning in UBLOX_AT_CellularNetwork.cpp
diff --git a/features/cellular/framework/targets/UBLOX/AT/UBLOX_AT_CellularNetwork.cpp b/features/cellular/framework/targets/UBLOX/AT/UBLOX_AT_CellularNetwork.cpp
@@ -33,6 +33,8 @@ UBLOX_AT_CellularNetwork::~UBLOX_AT_CellularNetwork()
 
 nsapi_error_t UBLOX_AT_CellularNetwork::set_access_technology_impl(RadioAccessTechnology opRat)
 {
+    nsapi_error_t ret = NSAPI_ERROR_OK;
+
     switch (opRat) {
 #if defined(TARGET_UBLOX_C030_U201) || defined(TARGET_UBLOX_C027)
         case RAT_GSM:
@@ -58,9 +60,9 @@ nsapi_error_t UBLOX_AT_CellularNetwork::set_access_technology_impl(RadioAccessTe
 #endif
         default: {
             _op_act = RAT_UNKNOWN;
-            return NSAPI_ERROR_UNSUPPORTED;
+            ret = NSAPI_ERROR_UNSUPPORTED;
         }
     }
 
-    return NSAPI_ERROR_OK;
+    return(ret);
 }

Original file line number	Diff line number	Diff line change
`@@ -33,6 +33,8 @@ UBLOX_AT_CellularNetwork::~UBLOX_AT_CellularNetwork()`
`33`	`33`
`34`	`34`	`nsapi_error_t UBLOX_AT_CellularNetwork::set_access_technology_impl(RadioAccessTechnology opRat)`
`35`	`35`	`{`
	`36`	`+ nsapi_error_t ret = NSAPI_ERROR_OK;`
	`37`	`+`
`36`	`38`	`switch (opRat) {`
`37`	`39`	`#if defined(TARGET_UBLOX_C030_U201) \|\| defined(TARGET_UBLOX_C027)`
`38`	`40`	`case RAT_GSM:`
`@@ -58,9 +60,9 @@ nsapi_error_t UBLOX_AT_CellularNetwork::set_access_technology_impl(RadioAccessTe`
`58`	`60`	`#endif`
`59`	`61`	`default: {`
`60`	`62`	`_op_act = RAT_UNKNOWN;`
`61`		`- return NSAPI_ERROR_UNSUPPORTED;`
	`63`	`+ ret = NSAPI_ERROR_UNSUPPORTED;`
`62`	`64`	`}`
`63`	`65`	`}`
`64`	`66`
`65`		`- return NSAPI_ERROR_OK;`
	`67`	`+ return(ret);`
`66`	`68`	`}`