Changeset 8365 in josm for trunk


Ignore:
Timestamp:
2015-05-16T15:13:38+02:00 (4 years ago)
Author:
Don-vip
Message:

fix Findbugs performance issues

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

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/PurgeAction.java

    r8338 r8365  
    245245                    if (type != 0)
    246246                        return type;
    247                     return (Long.valueOf(o1.getUniqueId())).compareTo(o2.getUniqueId());
     247                    return Long.compare(o1.getUniqueId(), o2.getUniqueId());
    248248                }
    249249            });
  • trunk/src/org/openstreetmap/josm/data/APIDataSet.java

    r8338 r8365  
    338338                        @Override
    339339                        public int compare(Relation o1, Relation o2) {
    340                             return Integer.valueOf(uploadOrder.indexOf(o1)).compareTo(uploadOrder.indexOf(o2));
     340                            return Integer.compare(uploadOrder.indexOf(o1), uploadOrder.indexOf(o2));
    341341                        }
    342342                    }
  • trunk/src/org/openstreetmap/josm/data/imagery/ImageryInfo.java

    r8349 r8365  
    678678                this.imageryType = type;
    679679                if (m.group(2) != null) {
    680                     defaultMaxZoom = Integer.valueOf(m.group(2));
     680                    defaultMaxZoom = Integer.parseInt(m.group(2));
    681681                }
    682682                if (m.group(1) != null) {
    683                     defaultMinZoom = Integer.valueOf(m.group(1));
     683                    defaultMinZoom = Integer.parseInt(m.group(1));
    684684                }
    685685                break;
  • trunk/src/org/openstreetmap/josm/data/osm/Changeset.java

    r8130 r8365  
    9393
    9494    public int compareTo(Changeset other) {
    95         return Integer.valueOf(getId()).compareTo(other.getId());
     95        return Integer.compare(getId(), other.getId());
    9696    }
    9797
  • trunk/src/org/openstreetmap/josm/data/osm/Node.java

    r8291 r8365  
    296296    @Override
    297297    public int compareTo(OsmPrimitive o) {
    298         return o instanceof Node ? Long.valueOf(getUniqueId()).compareTo(o.getUniqueId()) : 1;
     298        return o instanceof Node ? Long.compare(getUniqueId(), o.getUniqueId()) : 1;
    299299    }
    300300
  • trunk/src/org/openstreetmap/josm/data/osm/NodeData.java

    r6890 r8365  
    1515    private double lon = Double.NaN;
    1616
    17     public NodeData() {}
     17    /**
     18     * Constructs a new {@code NodeData}.
     19     */
     20    public NodeData() {
    1821
     22    }
     23
     24    /**
     25     * Constructs a new {@code NodeData}.
     26     * @param data node data to copy
     27     */
    1928    public NodeData(NodeData data) {
    2029        super(data);
     
    4453    @Override
    4554    public EastNorth getEastNorth() {
    46         /*
    47          * No internal caching of projected coordinates needed. In contrast to getEastNorth()
    48          * on Node, this method is rarely used. Caching would be overkill.
    49          */
     55        // No internal caching of projected coordinates needed. In contrast to getEastNorth()
     56        // on Node, this method is rarely used. Caching would be overkill.
    5057        return Projections.project(getCoor());
    5158    }
     
    5360    @Override
    5461    public void setEastNorth(EastNorth eastNorth) {
    55         LatLon ll = Projections.inverseProject(eastNorth);
    56         setCoor(ll);
     62        setCoor(Projections.inverseProject(eastNorth));
    5763    }
    5864
     
    7682        visitor.visit(this);
    7783    }
    78 
    7984}
  • trunk/src/org/openstreetmap/josm/data/osm/NoteData.java

    r8338 r8365  
    5252                return -1;
    5353            }
    54             return Long.valueOf(Math.abs(n1.getId())).compareTo(Long.valueOf(Math.abs(n2.getId())));
     54            return Long.compare(Math.abs(n1.getId()), Math.abs(n2.getId()));
    5555        }
    5656    };
  • trunk/src/org/openstreetmap/josm/data/osm/Relation.java

    r8338 r8365  
    300300    @Override
    301301    public int compareTo(OsmPrimitive o) {
    302         return o instanceof Relation ? Long.valueOf(getUniqueId()).compareTo(o.getUniqueId()) : -1;
    303     }
    304 
     302        return o instanceof Relation ? Long.compare(getUniqueId(), o.getUniqueId()) : -1;
     303    }
     304
     305    /**
     306     * Returns the first member.
     307     * @return first member, or {@code null}
     308     */
    305309    public RelationMember firstMember() {
    306         if (isIncomplete()) return null;
    307 
    308         RelationMember[] members = this.members;
    309         return (members.length == 0) ? null : members[0];
    310     }
     310        return (isIncomplete() || members.length == 0) ? null : members[0];
     311    }
     312
     313    /**
     314     * Returns the last member.
     315     * @return last member, or {@code null}
     316     */
    311317    public RelationMember lastMember() {
    312         if (isIncomplete()) return null;
    313 
    314         RelationMember[] members = this.members;
    315         return (members.length == 0) ? null : members[members.length - 1];
     318        return (isIncomplete() || members.length == 0) ? null : members[members.length - 1];
    316319    }
    317320
  • trunk/src/org/openstreetmap/josm/data/osm/Way.java

    r8345 r8365  
    351351        if (o instanceof Relation)
    352352            return 1;
    353         return o instanceof Way ? Long.valueOf(getUniqueId()).compareTo(o.getUniqueId()) : -1;
     353        return o instanceof Way ? Long.compare(getUniqueId(), o.getUniqueId()) : -1;
    354354    }
    355355
  • trunk/src/org/openstreetmap/josm/data/osm/history/HistoryOsmPrimitive.java

    r8291 r8365  
    156156        if (this.id != o.id)
    157157            throw new ClassCastException(tr("Cannot compare primitive with ID ''{0}'' to primitive with ID ''{1}''.", o.id, this.id));
    158         return Long.valueOf(this.version).compareTo(o.version);
     158        return Long.compare(this.version, o.version);
    159159    }
    160160
  • trunk/src/org/openstreetmap/josm/data/validation/tests/DuplicateRelation.java

    r8346 r8365  
    131131     * Class to store relation data (keys are usually cleanup and may not be equal to original relation)
    132132     */
    133     private class RelationPair {
     133    private static class RelationPair {
    134134        /** Member objects of the relation */
    135135        private RelationMembers members;
  • trunk/src/org/openstreetmap/josm/data/validation/tests/SimilarNamedWays.java

    r8338 r8365  
    223223    }
    224224
    225     public class RegExprRule implements NormalizeRule {
     225    public static class RegExprRule implements NormalizeRule {
    226226        private final Pattern regExpr;
    227227        private final String replacement;
     
    243243    }
    244244
    245     public class SynonymRule implements NormalizeRule {
     245    public static class SynonymRule implements NormalizeRule {
    246246
    247247        private final String[] words;
  • trunk/src/org/openstreetmap/josm/gui/MainMenu.java

    r8318 r8365  
    950950     * When user presses Enter key then selected item of "searchResultsMenu" is triggered.
    951951     */
    952     private class SearchFieldKeyListener implements KeyListener {
     952    private static class SearchFieldKeyListener implements KeyListener {
    953953
    954954        @Override
     
    970970
    971971        @Override
    972         public void keyTyped(KeyEvent e) { }
    973 
    974         @Override
    975         public void keyReleased(KeyEvent e) { }
     972        public void keyTyped(KeyEvent e) {
     973            // Not used
     974        }
     975
     976        @Override
     977        public void keyReleased(KeyEvent e) {
     978            // Not used
     979        }
    976980    }
    977981
  • trunk/src/org/openstreetmap/josm/gui/MapView.java

    r8318 r8365  
    509509        }
    510510        // sort according to position in the list of layers, with one exception:
    511         // an active data layer always becomes a higher Z-Order than all other
    512         // data layers
    513         //
     511        // an active data layer always becomes a higher Z-Order than all other data layers
    514512        Collections.sort(
    515513                ret,
    516514                new Comparator<Layer>() {
    517                     @Override public int compare(Layer l1, Layer l2) {
     515                    @Override
     516                    public int compare(Layer l1, Layer l2) {
    518517                        if (l1 instanceof OsmDataLayer && l2 instanceof OsmDataLayer) {
    519518                            if (l1 == getActiveLayer()) return -1;
    520519                            if (l2 == getActiveLayer()) return 1;
    521                             return Integer.valueOf(layers.indexOf(l1)).compareTo(layers.indexOf(l2));
     520                            return Integer.compare(layers.indexOf(l1), layers.indexOf(l2));
    522521                        } else
    523                             return Integer.valueOf(layers.indexOf(l1)).compareTo(layers.indexOf(l2));
     522                            return Integer.compare(layers.indexOf(l1), layers.indexOf(l2));
    524523                    }
    525524                }
     
    540539     * Draw the component.
    541540     */
    542     @Override public void paint(Graphics g) {
     541    @Override
     542    public void paint(Graphics g) {
    543543        if (initialViewport != null) {
    544544            zoomTo(initialViewport);
  • trunk/src/org/openstreetmap/josm/gui/actionsupport/DeleteFromRelationConfirmationDialog.java

    r8338 r8365  
    213213                            cmp = o1.getParent().getDisplayName(nf).compareTo(o2.getParent().getDisplayName(nf));
    214214                            if (cmp != 0) return cmp;
    215                             return Integer.valueOf(o1.getPosition()).compareTo(o2.getPosition());
     215                            return Integer.compare(o1.getPosition(), o2.getPosition());
    216216                        }
    217217                    }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/NotesDialog.java

    r8318 r8365  
    222222    }
    223223
    224     private class NoteRenderer implements ListCellRenderer<Note> {
     224    private static class NoteRenderer implements ListCellRenderer<Note> {
    225225
    226226        private DefaultListCellRenderer defaultListCellRenderer = new DefaultListCellRenderer();
  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesCellRenderer.java

    r8189 r8365  
    4040        }
    4141    }
    42    
    43     @Override 
     42
     43    @Override
    4444    public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) {
    4545        Component c = super.getTableCellRendererComponent(table, value, isSelected, false, row, column);
     
    5353                Map<?, ?> v = (Map<?, ?>) value;
    5454                if (v.size() != 1) {    // Multiple values: give user a short summary of the values
    55                     Integer blank_count;
    56                     Integer other_count;
     55                    Integer blankCount;
     56                    Integer otherCount;
    5757                    if (v.get("") == null) {
    58                         blank_count = 0;
    59                         other_count = v.size();
     58                        blankCount = 0;
     59                        otherCount = v.size();
    6060                    } else {
    61                         blank_count = (Integer)v.get("");
    62                         other_count = v.size()-1;
     61                        blankCount = (Integer)v.get("");
     62                        otherCount = v.size()-1;
    6363                    }
    64                     str = "<";
    65                     if (other_count == 1) {
     64                    StringBuilder sb = new StringBuilder("<");
     65                    if (otherCount == 1) {
    6666                        for (Map.Entry<?, ?> entry : v.entrySet()) { // Find the non-blank value in the map
    6767                            if ( entry.getKey() != "") {
    6868                                /* I18n: properties display partial string joined with comma, frst is count, second is value */
    69                                 str += tr("{0} ''{1}''", entry.getValue().toString(), entry.getKey());
     69                                sb.append(tr("{0} ''{1}''", entry.getValue().toString(), entry.getKey()));
    7070                            }
    7171                        }
    7272                    } else {
    7373                        /* I18n: properties display partial string joined with comma */
    74                         str += trn("{0} different", "{0} different", other_count, other_count);
     74                        sb.append(trn("{0} different", "{0} different", otherCount, otherCount));
    7575                    }
    76                     if(blank_count > 0) {
     76                    if(blankCount > 0) {
    7777                        /* I18n: properties display partial string joined with comma */
    78                         str += trn(", {0} unset", ", {0} unset", blank_count, blank_count);
     78                        sb.append(trn(", {0} unset", ", {0} unset", blankCount, blankCount));
    7979                    }
    80                     str += ">";
     80                    sb.append(">");
     81                    str = sb.toString();
    8182                    c.setFont(c.getFont().deriveFont(Font.ITALIC));
    8283
  • trunk/src/org/openstreetmap/josm/gui/mappaint/NodeElemStyle.java

    r8345 r8365  
    44import java.awt.BasicStroke;
    55import java.awt.Color;
    6 import java.awt.Image;
    76import java.awt.Rectangle;
    87import java.awt.Stroke;
     
    2928    public final RotationAngle mapImageAngle;
    3029    public final Symbol symbol;
    31 
    32     private Image enabledNodeIcon;
    33     private Image disabledNodeIcon;
    34 
    35     private boolean enabledNodeIconIsTemporary;
    36     private boolean disabledNodeIconIsTemporary;
    3730
    3831    public enum SymbolShape { SQUARE, CIRCLE, TRIANGLE, PENTAGON, HEXAGON, HEPTAGON, OCTAGON, NONAGON, DECAGON }
  • trunk/src/org/openstreetmap/josm/gui/preferences/projection/CodeProjectionChoice.java

    r8308 r8365  
    8181                        int num1 = Integer.parseInt(matcher1.group(2));
    8282                        int num2 = Integer.parseInt(matcher2.group(2));
    83                         return Integer.valueOf(num1).compareTo(num2);
     83                        return Integer.compare(num1, num2);
    8484                    } else
    8585                        return -1;
  • trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPresetItems.java

    r8358 r8365  
    13751375            if (rows != null) {
    13761376                double height = renderer.getListCellRendererComponent(list,
    1377                         new PresetListEntry("x"), 0, false, false).getPreferredSize().getHeight() * Integer.valueOf(rows);
     1377                        new PresetListEntry("x"), 0, false, false).getPreferredSize().getHeight() * Integer.parseInt(rows);
    13781378                sp.setPreferredSize(new Dimension((int) sp.getPreferredSize().getWidth(), (int) height));
    13791379            }
  • trunk/src/org/openstreetmap/josm/tools/ImageOverlay.java

    r8290 r8365  
    7272        int height = -1;
    7373        if (offsetRight > 0 && offsetLeft > 0) {
    74             width = new Double(w*(offsetRight-offsetLeft)).intValue();
     74            width = (int) (w*(offsetRight-offsetLeft));
    7575        }
    7676        if (offsetTop > 0 && offsetBottom > 0) {
    77             width = new Double(h*(offsetBottom-offsetTop)).intValue();
     77            width = (int) (h*(offsetBottom-offsetTop));
    7878        }
    7979        ImageIcon overlay;
  • trunk/src/org/openstreetmap/josm/tools/WindowGeometry.java

    r8291 r8365  
    187187            final Matcher m = Pattern.compile("(\\d+)x(\\d+)(([+-])(\\d+)([+-])(\\d+))?").matcher(arg);
    188188            if (m.matches()) {
    189                 int w = Integer.valueOf(m.group(1));
    190                 int h = Integer.valueOf(m.group(2));
     189                int w = Integer.parseInt(m.group(1));
     190                int h = Integer.parseInt(m.group(2));
    191191                int x = screenDimension.x, y = screenDimension.y;
    192192                if (m.group(3) != null) {
    193                     x = Integer.valueOf(m.group(5));
    194                     y = Integer.valueOf(m.group(7));
     193                    x = Integer.parseInt(m.group(5));
     194                    y = Integer.parseInt(m.group(7));
    195195                    if ("-".equals(m.group(4))) {
    196196                        x = screenDimension.x + screenDimension.width - x - w;
Note: See TracChangeset for help on using the changeset viewer.