[MATH-336] Unnecessary null checks in equals methods

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.1
• Assignee: Unassigned
• Reporter: Sebb
• Created: 27/Jan/10 15:38
• Updated: 03/Apr/10 03:56
• Resolved: 14/Mar/10 01:35

Description

Many of the equals() methods have code like the following:

if (obj == null) {
    return false;
}
if (!(obj instanceof OpenMapRealVector)) {
    return false;
}

The null check is redundant, as null would cause the instanceof check to fail anyway, so the null check could just be removed.