Ignore:
Timestamp:
2013-10-07T20:18:17+02:00 (6 years ago)
Author:
Don-vip
Message:

Sonar/FindBugs - Loose coupling

Location:
trunk/src/org/openstreetmap/josm/gui/history
Files:
6 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserDialogManager.java

    r6223 r6316  
    99import java.util.Collection;
    1010import java.util.HashMap;
     11import java.util.List;
    1112import java.util.Map;
    1213
     
    102103     */
    103104    public void hideAll() {
    104         ArrayList<HistoryBrowserDialog> dialogs = new ArrayList<HistoryBrowserDialog>();
     105        List<HistoryBrowserDialog> dialogs = new ArrayList<HistoryBrowserDialog>();
    105106        dialogs.addAll(this.dialogs.values());
    106107        for (HistoryBrowserDialog dialog: dialogs) {
  • trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java

    r6105 r6316  
    88import java.util.Collections;
    99import java.util.HashSet;
     10import java.util.List;
    1011import java.util.Observable;
    1112
     
    553554    public class TagTableModel extends AbstractTableModel {
    554555
    555         private ArrayList<String> keys;
     556        private List<String> keys;
    556557        private PointInTimeType pointInTimeType;
    557558
  • trunk/src/org/openstreetmap/josm/gui/history/HistoryLoadTask.java

    r5926 r6316  
    1010import java.util.Collection;
    1111import java.util.HashSet;
     12import java.util.Set;
    1213
    1314import org.openstreetmap.josm.data.osm.OsmPrimitive;
     
    4849    private boolean canceled = false;
    4950    private Exception lastException  = null;
    50     private HashSet<PrimitiveId> toLoad;
     51    private Set<PrimitiveId> toLoad;
    5152    private HistoryDataSet loadedData;
    5253    private OsmServerHistoryReader reader = null;
  • trunk/src/org/openstreetmap/josm/gui/history/RelationMemberListTableCellRenderer.java

    r6084 r6316  
    77import java.awt.Component;
    88import java.util.HashMap;
     9import java.util.Map;
    910
    1011import javax.swing.ImageIcon;
     
    3031    public final static Color BGCOLOR_SELECTED = new Color(143,170,255);
    3132
    32     private HashMap<OsmPrimitiveType, ImageIcon> icons;
     33    private Map<OsmPrimitiveType, ImageIcon> icons;
    3334
    3435    public RelationMemberListTableCellRenderer(){
  • trunk/src/org/openstreetmap/josm/gui/history/SelectionSynchronizer.java

    r6084 r6316  
    33
    44import java.util.ArrayList;
     5import java.util.List;
    56
    67import javax.swing.DefaultListSelectionModel;
     
    1112public class SelectionSynchronizer implements ListSelectionListener {
    1213
    13     private ArrayList<ListSelectionModel> participants;
     14    private List<ListSelectionModel> participants;
    1415
     16    /**
     17     * Constructs a new {@code SelectionSynchronizer}.
     18     */
    1519    public SelectionSynchronizer() {
    1620        participants = new ArrayList<ListSelectionModel>();
  • trunk/src/org/openstreetmap/josm/gui/history/TwoColumnDiff.java

    r6246 r6316  
    55import java.awt.Color;
    66import java.util.ArrayList;
     7import java.util.List;
    78
    89import org.openstreetmap.josm.gui.history.TwoColumnDiff.Item.DiffItemType;
     
    5051    }
    5152
    52     public ArrayList<Item> referenceDiff;
    53     public ArrayList<Item> currentDiff;
     53    public List<Item> referenceDiff;
     54    public List<Item> currentDiff;
    5455    Object[] reference;
    5556    Object[] current;
Note: See TracChangeset for help on using the changeset viewer.