Ignore:
Timestamp:
2014-04-19T00:36:43+02:00 (6 years ago)
Author:
Don-vip
Message:

sonar - place Strings literals on the left side when checking for equality (can avoid NPEs)

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

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/projection/CustomProjection.java

    r6890 r6990  
    173173                String key = m.group(1);
    174174                // alias
    175                 if (key.equals("k")) {
     175                if ("k".equals(key)) {
    176176                    key = Param.k_0.key;
    177177                }
     
    181181                    // some aliases
    182182                    if (key.equals(Param.proj.key)) {
    183                         if (value.equals("longlat") || value.equals("latlon") || value.equals("latlong")) {
     183                        if ("longlat".equals(value) || "latlon".equals(value) || "latlong".equals(value)) {
    184184                            value = "lonlat";
    185185                        }
     
    264264                nadgridsId = nadgridsId.substring(1);
    265265            }
    266             if (nadgridsId.equals("null"))
     266            if ("null".equals(nadgridsId))
    267267                return new NullDatum(null, ellps);
    268268            NTV2GridShiftFileWrapper nadgrids = Projections.getNTV2Grid(nadgridsId);
  • trunk/src/org/openstreetmap/josm/data/projection/datum/NTV2GridShiftFile.java

    r6830 r6990  
    169169        HashMap<String, List<NTV2SubGrid>> subGridMap = new HashMap<String, List<NTV2SubGrid>>();
    170170        for (int i = 0; i < subGrid.length; i++) {
    171             if (subGrid[i].getParentSubGridName().equalsIgnoreCase("NONE")) {
     171            if ("NONE".equalsIgnoreCase(subGrid[i].getParentSubGridName())) {
    172172                topLevelCount++;
    173173            }
     
    177177        topLevelCount = 0;
    178178        for (int i = 0; i < subGrid.length; i++) {
    179             if (subGrid[i].getParentSubGridName().equalsIgnoreCase("NONE")) {
     179            if ("NONE".equalsIgnoreCase(subGrid[i].getParentSubGridName())) {
    180180                topLevelSubGrid[topLevelCount++] = subGrid[i];
    181181            } else {
Note: See TracChangeset for help on using the changeset viewer.