#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)) ) {

Change History (0)

Note: See TracTickets for help on using tickets.