#3301: src/c/src/lib/libbsp/sparc/shared/tmtc/grtc.c:1871]: (style) Suspicious
condition
------------------------------+--------------------
  Reporter:  David Binderman  |      Owner:  (none)
      Type:  defect           |     Status:  new
  Priority:  normal           |  Milestone:
 Component:  admin            |    Version:
  Severity:  normal           |   Keywords:
Blocked By:                   |   Blocking:
------------------------------+--------------------
 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)) ) {

--
Ticket URL: <http://devel.rtems.org/ticket/3301>
RTEMS Project <http://www.rtems.org/>
RTEMS Project
_______________________________________________
bugs mailing list
bugs@rtems.org
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to