Changeset 14098 in josm for trunk/src/org


Ignore:
Timestamp:
2018-08-05T23:04:41+02:00 (6 years ago)
Author:
Don-vip
Message:

fix PMD warnings

Location:
trunk/src/org/openstreetmap/josm
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/osm/search/SearchCompiler.java

    r14093 r14098  
    12201220                return false;
    12211221            ExactType other = (ExactType) obj;
    1222             if (type != other.type)
    1223                 return false;
    1224             return true;
     1222            return type == other.type;
    12251223        }
    12261224    }
     
    13921390                return false;
    13931391            Nth other = (Nth) obj;
    1394             if (modulo != other.modulo)
    1395                 return false;
    1396             if (nth != other.nth)
    1397                 return false;
    1398             return true;
     1392            return modulo == other.modulo
     1393                   && nth == other.nth;
    13991394        }
    14001395    }
     
    14511446                return false;
    14521447            RangeMatch other = (RangeMatch) obj;
    1453             if (max != other.max)
    1454                 return false;
    1455             if (min != other.min)
    1456                 return false;
    1457             return true;
     1448            return max == other.max
     1449                && min == other.min;
    14581450        }
    14591451    }
     
    18591851                return false;
    18601852            InArea other = (InArea) obj;
    1861             if (all != other.all)
    1862                 return false;
    1863             return true;
     1853            return all == other.all;
    18641854        }
    18651855    }
  • trunk/src/org/openstreetmap/josm/tools/template_engine/CompoundTemplateEntry.java

    r14093 r14098  
    6969            return false;
    7070        CompoundTemplateEntry other = (CompoundTemplateEntry) obj;
    71         if (!Arrays.equals(entries, other.entries))
    72             return false;
    73         return true;
     71        return Arrays.equals(entries, other.entries);
    7472    }
    7573}
Note: See TracChangeset for help on using the changeset viewer.