Opened on 02/14/18 at 15:20:52
#3301 new defect
src/c/src/lib/libbsp/sparc/shared/tmtc/grtc.c:1871]: (style) Suspicious condition
Reported by: | David Binderman | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | admin | Version: | |
Severity: | normal | Keywords: | |
Cc: | Blocked By: | ||
Blocking: |
Description
src/c/src/lib/libbsp/sparc/shared/tmtc/grtc.c:1871]: (style) Suspicious condition (assignment + comparison); Clarify expression with parentheses.
Source code is
while ( pDev->running && ((ret=process_dma(pDev) == 2) (ret == -1)) ) {
maybe better code
while ( pDev->running && (((ret = process_dma(pDev)) == 2) (ret == -1)) ) {
Note: See
TracTickets for help on using
tickets.