gsm/cdma: unsigned integers can't be < 0, so remove unnecessary conditions
It fixes compilation with the llvm/clang compiler as well.
This commit is contained in:
committed by
Jiří Klimeš
parent
2a9a6ad79b
commit
4c514bfd6d
@@ -591,7 +591,7 @@ crm_done (MMAtSerialPort *port,
|
|||||||
if (p) {
|
if (p) {
|
||||||
errno = 0;
|
errno = 0;
|
||||||
num = strtoul (p, NULL, 10);
|
num = strtoul (p, NULL, 10);
|
||||||
if (num >= 0 && num <= 4 && (errno == 0)) {
|
if (num <= 4 && (errno == 0)) {
|
||||||
MM_GENERIC_CDMA_GET_PRIVATE (user_data)->orig_crm = (guint32) num;
|
MM_GENERIC_CDMA_GET_PRIVATE (user_data)->orig_crm = (guint32) num;
|
||||||
MM_GENERIC_CDMA_GET_PRIVATE (user_data)->cur_crm = (guint32) num;
|
MM_GENERIC_CDMA_GET_PRIVATE (user_data)->cur_crm = (guint32) num;
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -2892,10 +2892,8 @@ handle_reg_status_response (MMGenericGsm *self,
|
|||||||
if (act != -1)
|
if (act != -1)
|
||||||
mm_generic_gsm_update_access_technology (self, etsi_act_to_mm_act (act));
|
mm_generic_gsm_update_access_technology (self, etsi_act_to_mm_act (act));
|
||||||
|
|
||||||
if (status >= 0) {
|
/* Update cached registration status */
|
||||||
/* Update cached registration status */
|
reg_status_updated (self, cgreg_to_reg_type (cgreg), status, NULL);
|
||||||
reg_status_updated (self, cgreg_to_reg_type (cgreg), status, NULL);
|
|
||||||
}
|
|
||||||
|
|
||||||
return TRUE;
|
return TRUE;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user