Fix cppcheck warnings for alinco

pull/154/head
Michael Black 2019-12-06 14:59:42 -06:00
rodzic 3be393c015
commit 3b581bdc6b
Nie znaleziono w bazie danych klucza dla tego podpisu
ID klucza GPG: 6599353EC683404D
2 zmienionych plików z 7 dodań i 7 usunięć

Wyświetl plik

@ -138,7 +138,8 @@ int alinco_transaction(RIG *rig,
return retval; return retval;
} }
retval -= 2; if (retval > 2) { retval -= 2; }
echobuf[retval] = 0; echobuf[retval] = 0;
if (strcmp(echobuf, "OK") == 0) if (strcmp(echobuf, "OK") == 0)
@ -695,7 +696,7 @@ int alinco_get_func(RIG *rig, vfo_t vfo, setting_t func, int *status)
funcbuf[2] = '\0'; funcbuf[2] = '\0';
settings = strtol(funcbuf, (char **)NULL, 16); settings = strtol(funcbuf, (char **)NULL, 16);
*status = settings & 0x08 ? 1 : 0; *status = (settings & 0x08) ? 1 : 0;
break; break;
case RIG_FUNC_FAGC: case RIG_FUNC_FAGC:
@ -708,7 +709,7 @@ int alinco_get_func(RIG *rig, vfo_t vfo, setting_t func, int *status)
funcbuf[2] = '\0'; funcbuf[2] = '\0';
settings = strtol(funcbuf, (char **)NULL, 16); settings = strtol(funcbuf, (char **)NULL, 16);
*status = settings & 0x01 ? 1 : 0; *status = (settings & 0x01) ? 1 : 0;
break; break;
case RIG_FUNC_NB: case RIG_FUNC_NB:
@ -721,7 +722,7 @@ int alinco_get_func(RIG *rig, vfo_t vfo, setting_t func, int *status)
funcbuf[2] = '\0'; funcbuf[2] = '\0';
settings = strtol(funcbuf, (char **)NULL, 16); settings = strtol(funcbuf, (char **)NULL, 16);
*status = settings & 0x04 ? 1 : 0; *status = (settings & 0x04) ? 1 : 0;
break; break;
default: default:
@ -1054,8 +1055,7 @@ int alinco_set_ctcss_tone(RIG *rig, vfo_t vfo, tone_t tone)
caps = rig->caps; caps = rig->caps;
/* TODO: replace 200 by something like RIGTONEMAX */ for (i = 0; caps->ctcss_list[i] != 0; i++)
for (i = 0; caps->ctcss_list[i] != 0 && i < 200; i++)
{ {
if (caps->ctcss_list[i] == tone) if (caps->ctcss_list[i] == tone)
{ {

Wyświetl plik

@ -79,7 +79,7 @@ const struct rig_caps dx77_caps =
.rig_model = RIG_MODEL_DX77, .rig_model = RIG_MODEL_DX77,
.model_name = "DX-77", .model_name = "DX-77",
.mfg_name = "Alinco", .mfg_name = "Alinco",
.version = "0.7", .version = "0.8",
.copyright = "LGPL", .copyright = "LGPL",
.status = RIG_STATUS_BETA, .status = RIG_STATUS_BETA,
.rig_type = RIG_TYPE_TRANSCEIVER, .rig_type = RIG_TYPE_TRANSCEIVER,