Ignore:
Timestamp:
2017-04-13T01:08:58+02:00 (9 years ago)
Author:
Don-vip
Message:

sonar - squid:S1126 - Return of boolean expressions should not be wrapped into an "if-then-else" statement

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/util/RotationAngle.java

    r11731 r11893  
    5757                return true;
    5858            }
    59             if (obj == null) {
    60                 return false;
    61             }
    62             if (getClass() != obj.getClass()) {
    63                 return false;
    64             }
    65             return true;
     59            return obj != null && getClass() == obj.getClass();
    6660        }
    67 
    6861    }
    6962
    … …  
    9285            final int prime = 31;
    9386            int result = 1;
    94             long temp;
    95             temp = Double.doubleToLongBits(angle);
     87            long temp = Double.doubleToLongBits(angle);
    9688            result = prime * result + (int) (temp ^ (temp >>> 32));
    9789            return result;
    … …  
    10395                return true;
    10496            }
    105             if (obj == null) {
    106                 return false;
    107             }
    108             if (getClass() != obj.getClass()) {
     97            if (obj == null || getClass() != obj.getClass()) {
    10998                return false;
    11099            }
    111100            StaticRotationAngle other = (StaticRotationAngle) obj;
    112             if (Double.doubleToLongBits(angle) != Double.doubleToLongBits(other.angle)) {
    113                 return false;
    114             }
    115             return true;
     101            return Double.doubleToLongBits(angle) == Double.doubleToLongBits(other.angle);
    116102        }
    117103    }
Note: See TracChangeset for help on using the changeset viewer.