Menu

#9 [gtkextra.override:776]: (style) Same expression on both sides of '||'.

CVS
closed-accepted
None
5
2014-12-04
2014-02-12
dcb
No

Source code is

if ( (rx < 0) || (ry < 0) || (rdx < 0) || (rdy < 0)
 || (rz < 0) || (rdz < 0) || (rda < 0) || (rda < 0) ) {

Maybe better code might be

 if ( (rx < 0) || (ry < 0) || (rdx < 0) || (rdy < 0)
 || (rz < 0) || (rdz < 0) || (ra < 0) || (rda < 0) ) {

Discussion

  • Roy Rankin

    Roy Rankin - 2014-12-04
    • status: open --> closed-accepted
    • assigned_to: Roy Rankin
     
  • Roy Rankin

    Roy Rankin - 2014-12-04

    Applied

     

Log in to post a comment.