CANopen: static analysis: side effects on right hand of logical operator, '||' [MISRA 2012 Rule 13.5, required]
This commit is contained in:
parent
4cee088192
commit
926dc62873
1 changed files with 5 additions and 3 deletions
|
|
@ -1450,6 +1450,9 @@ CO_NMT_reset_cmd_t CO_process(CO_t *co,
|
|||
#endif
|
||||
|
||||
if (CO_GET_CNT(LEDS) == 1U) {
|
||||
bool_t ErrSync = CO_isError(co->em, CO_EM_SYNC_TIME_OUT);
|
||||
bool_t ErrHbCons = CO_isError(co->em, CO_EM_HEARTBEAT_CONSUMER);
|
||||
bool_t ErrHbConsRemote = CO_isError(co->em, CO_EM_HB_CONSUMER_REMOTE_RESET);
|
||||
CO_LEDs_process(co->LEDs,
|
||||
timeDifference_us,
|
||||
unc ? CO_NMT_INITIALIZING : NMTstate,
|
||||
|
|
@ -1457,9 +1460,8 @@ CO_NMT_reset_cmd_t CO_process(CO_t *co,
|
|||
(CANerrorStatus & CO_CAN_ERRTX_BUS_OFF) != 0U,
|
||||
(CANerrorStatus & CO_CAN_ERR_WARN_PASSIVE) != 0U,
|
||||
false, /* RPDO event timer timeout */
|
||||
unc ? false : CO_isError(co->em, CO_EM_SYNC_TIME_OUT),
|
||||
unc ? false : (CO_isError(co->em, CO_EM_HEARTBEAT_CONSUMER)
|
||||
|| CO_isError(co->em, CO_EM_HB_CONSUMER_REMOTE_RESET)),
|
||||
unc ? false : ErrSync,
|
||||
unc ? false : (ErrHbCons || ErrHbConsRemote),
|
||||
CO_getErrorRegister(co->em) != 0U,
|
||||
CO_STATUS_FIRMWARE_DOWNLOAD_IN_PROGRESS,
|
||||
timerNext_us);
|
||||
|
|
|
|||
Loading…
Reference in a new issue