Changeset 10619 in josm for trunk/src


Ignore:
Timestamp:
2016-07-24T00:19:35+02:00 (8 years ago)
Author:
Don-vip
Message:

see #11390 - Java 8: use List.sort(Comparator) instead of Collections.sort(list, Comparator)

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/ExtensionFileFilter.java

    r10463 r10619  
    119119
    120120    protected static void sort(List<ExtensionFileFilter> filters) {
    121         Collections.sort(
    122                 filters,
    123                 new Comparator<ExtensionFileFilter>() {
    124                     private AllFormatsImporter all = new AllFormatsImporter();
    125                     @Override
    126                     public int compare(ExtensionFileFilter o1, ExtensionFileFilter o2) {
    127                         if (o1.getDescription().equals(all.filter.getDescription())) return 1;
    128                         if (o2.getDescription().equals(all.filter.getDescription())) return -1;
    129                         return o1.getDescription().compareTo(o2.getDescription());
    130                     }
     121        filters.sort(new Comparator<ExtensionFileFilter>() {
     122                private AllFormatsImporter all = new AllFormatsImporter();
     123                @Override
     124                public int compare(ExtensionFileFilter o1, ExtensionFileFilter o2) {
     125                    if (o1.getDescription().equals(all.filter.getDescription())) return 1;
     126                    if (o2.getDescription().equals(all.filter.getDescription())) return -1;
     127                    return o1.getDescription().compareTo(o2.getDescription());
    131128                }
     129            }
    132130        );
    133131    }

  • trunk/src/org/openstreetmap/josm/actions/JoinNodeWayAction.java

    r10467 r10619  
    130130            List<Integer> segmentIndexes = new LinkedList<>();
    131131            segmentIndexes.addAll(innerEntry.keySet());
    132             Collections.sort(segmentIndexes, Collections.reverseOrder());
     132            segmentIndexes.sort(Collections.reverseOrder());
    133133
    134134            List<Node> wayNodes = w.getNodes();
    … …  
    149149                List<Node> nodesToAdd = new LinkedList<>();
    150150                nodesToAdd.addAll(nodesInSegment);
    151                 Collections.sort(nodesToAdd, new NodeDistanceToRefNodeComparator(
     151                nodesToAdd.sort(new NodeDistanceToRefNodeComparator(
    152152                        w.getNode(segmentIndex), w.getNode(segmentIndex+1), !joinWayToNode));
    153153                wayNodes.addAll(segmentIndex + 1, nodesToAdd);

  • trunk/src/org/openstreetmap/josm/actions/PurgeAction.java

    r10601 r10619  
    1313import java.util.ArrayList;
    1414import java.util.Collection;
    15 import java.util.Collections;
    1615import java.util.HashSet;
    1716import java.util.List;
    … …  
    244243                            ImageProvider.get("warning-small"), JLabel.LEFT), GBC.eol().fill(GBC.HORIZONTAL));
    245244
    246             Collections.sort(toPurgeAdditionally, (o1, o2) -> {
     245            toPurgeAdditionally.sort((o1, o2) -> {
    247246                int type = o2.getType().compareTo(o1.getType());
    248247                if (type != 0)

  • trunk/src/org/openstreetmap/josm/data/APIDataSet.java

    r10608 r10619  
    44import java.util.ArrayList;
    55import java.util.Collection;
    6 import java.util.Collections;
    76import java.util.HashMap;
    87import java.util.HashSet;
    … …  
    7271        }
    7372        OsmPrimitiveComparator c = new OsmPrimitiveComparator(false, true);
    74         Collections.sort(toDelete, c);
    75         Collections.sort(toAdd, c);
    76         Collections.sort(toUpdate, c);
     73        toDelete.sort(c);
     74        toAdd.sort(c);
     75        toUpdate.sort(c);
    7776    }
    7877
    … …  
    332331            }
    333332            List<Relation> ret = new ArrayList<>(relations);
    334             Collections.sort(
    335                     ret,
    336                     (o1, o2) -> Integer.compare(uploadOrder.indexOf(o1), uploadOrder.indexOf(o2))
    337                     );
     333            ret.sort((o1, o2) -> Integer.compare(uploadOrder.indexOf(o1), uploadOrder.indexOf(o2)));
    338334            return ret;
    339335        }

  • trunk/src/org/openstreetmap/josm/data/osm/NoteData.java

    r10608 r10619  
    102102    public Collection<Note> getSortedNotes() {
    103103        final List<Note> list = new ArrayList<>(noteList);
    104         Collections.sort(list, comparator);
     104        list.sort(comparator);
    105105        return list;
    106106    }

  • trunk/src/org/openstreetmap/josm/data/osm/history/History.java

    r10608 r10619  
    44import java.text.MessageFormat;
    55import java.util.ArrayList;
    6 import java.util.Collections;
    76import java.util.Date;
    87import java.util.List;
    … …  
    6968    public History sortAscending() {
    7069        List<HistoryOsmPrimitive> copy = new ArrayList<>(versions);
    71         Collections.sort(copy, (o1, o2) -> o1.compareTo(o2));
     70        copy.sort((o1, o2) -> o1.compareTo(o2));
    7271        return new History(id, type, copy);
    7372    }
    … …  
    7978    public History sortDescending() {
    8079        List<HistoryOsmPrimitive> copy = new ArrayList<>(versions);
    81         Collections.sort(copy, (o1, o2) -> o2.compareTo(o1));
     80        copy.sort((o1, o2) -> o2.compareTo(o1));
    8281        return new History(id, type, copy);
    8382    }

  • trunk/src/org/openstreetmap/josm/gui/ImageryMenu.java

    r10611 r10619  
    1111import java.util.ArrayList;
    1212import java.util.Collection;
    13 import java.util.Collections;
    1413import java.util.Comparator;
    1514import java.util.Iterator;
    … …  
    150149        // for each configured ImageryInfo, add a menu entry.
    151150        final List<ImageryInfo> savedLayers = new ArrayList<>(ImageryLayerInfo.instance.getLayers());
    152         Collections.sort(savedLayers, alphabeticImageryComparator);
     151        savedLayers.sort(alphabeticImageryComparator);
    153152        for (final ImageryInfo u : savedLayers) {
    154153            addDynamic(new AddImageryLayerAction(u));
    … …  
    184183            }
    185184            if (!inViewLayers.isEmpty()) {
    186                 Collections.sort(inViewLayers, alphabeticImageryComparator);
     185                inViewLayers.sort(alphabeticImageryComparator);
    187186                addDynamicSeparator();
    188187                for (ImageryInfo i : inViewLayers) {

  • trunk/src/org/openstreetmap/josm/gui/conflict/tags/TagConflictResolverModel.java

    r10611 r10619  
    55import java.beans.PropertyChangeSupport;
    66import java.util.ArrayList;
    7 import java.util.Collections;
    87import java.util.HashMap;
    98import java.util.HashSet;
    … …  
    6564
    6665    protected void sort() {
    67         Collections.sort(
    68                 displayedKeys,
    69                 (key1, key2) -> {
    70                     if (decisions.get(key1).isDecided() && !decisions.get(key2).isDecided())
    71                         return 1;
    72                     else if (!decisions.get(key1).isDecided() && decisions.get(key2).isDecided())
    73                         return -1;
    74                     return key1.compareTo(key2);
    75                 }
     66        displayedKeys.sort((key1, key2) -> {
     67                if (decisions.get(key1).isDecided() && !decisions.get(key2).isDecided())
     68                    return 1;
     69                else if (!decisions.get(key1).isDecided() && decisions.get(key2).isDecided())
     70                    return -1;
     71                return key1.compareTo(key2);
     72            }
    7673        );
    7774    }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/DeleteFromRelationConfirmationDialog.java

    r10465 r10619  
    1414import java.util.ArrayList;
    1515import java.util.Collection;
    16 import java.util.Collections;
    1716import java.util.Comparator;
    1817import java.util.HashSet;
    … …  
    202201
    203202        protected void sort() {
    204             Collections.sort(
    205                     data,
    206                     new Comparator<RelationToChildReference>() {
    207                         private NameFormatter nf = DefaultNameFormatter.getInstance();
    208                         @Override
    209                         public int compare(RelationToChildReference o1, RelationToChildReference o2) {
    210                             int cmp = o1.getChild().getDisplayName(nf).compareTo(o2.getChild().getDisplayName(nf));
    211                             if (cmp != 0) return cmp;
    212                             cmp = o1.getParent().getDisplayName(nf).compareTo(o2.getParent().getDisplayName(nf));
    213                             if (cmp != 0) return cmp;
    214                             return Integer.compare(o1.getPosition(), o2.getPosition());
    215                         }
     203            data.sort(new Comparator<RelationToChildReference>() {
     204                    private NameFormatter nf = DefaultNameFormatter.getInstance();
     205                    @Override
     206                    public int compare(RelationToChildReference o1, RelationToChildReference o2) {
     207                        int cmp = o1.getChild().getDisplayName(nf).compareTo(o2.getChild().getDisplayName(nf));
     208                        if (cmp != 0) return cmp;
     209                        cmp = o1.getParent().getDisplayName(nf).compareTo(o2.getParent().getDisplayName(nf));
     210                        if (cmp != 0) return cmp;
     211                        return Integer.compare(o1.getPosition(), o2.getPosition());
    216212                    }
     213                }
    217214            );
    218215        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/InspectPrimitiveDialog.java

    r10611 r10619  
    1010import java.util.ArrayList;
    1111import java.util.Collection;
    12 import java.util.Collections;
    1312import java.util.List;
    1413import java.util.Locale;
    … …  
    107106    protected static String buildDataText(OsmDataLayer layer, List<OsmPrimitive> primitives) {
    108107        InspectPrimitiveDataText dt = new InspectPrimitiveDataText(layer);
    109         Collections.sort(primitives, new OsmPrimitiveComparator());
     108        primitives.sort(new OsmPrimitiveComparator());
    110109        for (OsmPrimitive o : primitives) {
    111110            dt.addPrimitive(o);

  • trunk/src/org/openstreetmap/josm/gui/dialogs/RelationListDialog.java

    r10611 r10619  
    395395
    396396        public void sort() {
    397             Collections.sort(
    398                     relations,
    399                     DefaultNameFormatter.getInstance().getRelationComparator()
    400                     );
     397            relations.sort(DefaultNameFormatter.getInstance().getRelationComparator());
    401398        }
    402399

  • trunk/src/org/openstreetmap/josm/gui/dialogs/SelectionListDialog.java

    r10611 r10619  
    657657         */
    658658        public synchronized void sort() {
    659             if (this.selection.size() <= Main.pref.getInteger("selection.no_sort_above", 100000)) {
    660                 boolean quick = this.selection.size() > Main.pref.getInteger("selection.fast_sort_above", 10000);
    661                 Collections.sort(this.selection, new OsmPrimitiveComparator(quick, false));
     659            if (selection.size() <= Main.pref.getInteger("selection.no_sort_above", 100000)) {
     660                boolean quick = selection.size() > Main.pref.getInteger("selection.fast_sort_above", 10000);
     661                selection.sort(new OsmPrimitiveComparator(quick, false));
    662662            }
    663663        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManagerModel.java

    r10611 r10619  
    66import java.util.ArrayList;
    77import java.util.Collection;
    8 import java.util.Collections;
    98import java.util.HashSet;
    109import java.util.List;
    … …  
    151150
    152151    protected void sort() {
    153         Collections.sort(
    154                 this.data,
    155                 (o1, o2) -> {
    156                     if (o1.getId() < o2.getId()) return 1;
    157                     if (o1.getId() == o2.getId()) return 0;
    158                     return -1;
    159                 }
     152        data.sort((o1, o2) -> {
     153                if (o1.getId() < o2.getId()) return 1;
     154                if (o1.getId() == o2.getId()) return 0;
     155                return -1;
     156            }
    160157        );
    161158    }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetContentTableModel.java

    r10611 r10619  
    33
    44import java.util.ArrayList;
    5 import java.util.Collections;
    65import java.util.HashSet;
    76import java.util.Iterator;
    … …  
    8079
    8180    protected void sort() {
    82         Collections.sort(
    83                 data,
    84                 (c1, c2) -> {
    85                     if (c1.getModificationType().equals(c2.getModificationType())) {
    86                         long id1 = c1.getPrimitive().getId();
    87                         long id2 = c2.getPrimitive().getId();
     81        data.sort((c1, c2) -> {
     82                if (c1.getModificationType().equals(c2.getModificationType())) {
     83                    long id1 = c1.getPrimitive().getId();
     84                    long id2 = c2.getPrimitive().getId();
    8885
    89                         if (id1 == id2)
    90                             return 0;
    91                         else if (id1 < id2)
    92                             return -1;
    93                         return 1;
     86                    if (id1 == id2)
     87                        return 0;
     88                    else if (id1 < id2)
     89                        return -1;
     90                    return 1;
     91                }
     92                switch(c1.getModificationType()) {
     93                case CREATED: return -1;
     94                case UPDATED:
     95                    switch(c2.getModificationType()) {
     96                    case CREATED: return 1;
     97                    default: return -1;
    9498                    }
    95                     switch(c1.getModificationType()) {
    96                     case CREATED: return -1;
    97                     case UPDATED:
    98                         switch(c2.getModificationType()) {
    99                         case CREATED: return 1;
    100                         default: return -1;
    101                         }
    102                     case DELETED:
    103                         return 1;
    104                     }
    105                     // should not happen
    106                     return 0;
     99                case DELETED:
     100                    return 1;
    107101                }
     102                // should not happen
     103                return 0;
     104            }
    108105        );
    109106    }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetListModel.java

    r10611 r10619  
    44import java.util.ArrayList;
    55import java.util.Collection;
    6 import java.util.Collections;
    76import java.util.HashSet;
    87import java.util.List;
    … …  
    140139
    141140    protected void sort() {
    142         Collections.sort(
    143                 data,
    144                 (cs1, cs2) -> {
    145                     if (cs1.getId() > cs2.getId()) return -1;
    146                     if (cs1.getId() == cs2.getId()) return 0;
    147                     return 1;
    148                 }
     141        data.sort((cs1, cs2) -> {
     142                if (cs1.getId() > cs2.getId()) return -1;
     143                if (cs1.getId() == cs2.getId()) return 0;
     144                return 1;
     145            }
    149146        );
    150147    }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesDialog.java

    r10611 r10619  
    702702
    703703        List<Relation> sortedRelations = new ArrayList<>(roles.keySet());
    704         Collections.sort(sortedRelations, (o1, o2) -> {
     704        sortedRelations.sort((o1, o2) -> {
    705705            int comp = Boolean.compare(o1.isDisabledAndHidden(), o2.isDisabledAndHidden());
    706706            return comp != 0 ? comp : DefaultNameFormatter.getInstance().getRelationComparator().compare(o1, o2);

  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/TagEditHelper.java

    r10611 r10619  
    415415            AutoCompletionManager autocomplete = Main.getLayerManager().getEditLayer().data.getAutoCompletionManager();
    416416            List<AutoCompletionListItem> keyList = autocomplete.getKeys();
    417             Collections.sort(keyList, defaultACItemComparator);
     417            keyList.sort(defaultACItemComparator);
    418418
    419419            keys = new AutoCompletingComboBox(key);
    … …  
    428428
    429429            List<AutoCompletionListItem> valueList = autocomplete.getValues(getAutocompletionKeys(key));
    430             Collections.sort(valueList, usedValuesAwareComparator);
     430            valueList.sort(usedValuesAwareComparator);
    431431
    432432            final String selection = m.size() != 1 ? tr("<different>") : m.entrySet().iterator().next().getKey();
    … …  
    612612
    613613                   List<AutoCompletionListItem> valueList = autocomplete.getValues(getAutocompletionKeys(key));
    614                    Collections.sort(valueList, comparator);
     614                   valueList.sort(comparator);
    615615                   if (Main.isTraceEnabled()) {
    616616                       Main.trace("Focus gained by {0}, e={1}", values, e);
    … …  
    687687            }
    688688
    689             Collections.sort(keyList, defaultACItemComparator);
     689            keyList.sort(defaultACItemComparator);
    690690            keys.setPossibleACItems(keyList);
    691691            keys.setEditable(true);

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/sort/RelationSorter.java

    r10611 r10619  
    44import java.util.ArrayList;
    55import java.util.Collection;
    6 import java.util.Collections;
    76import java.util.HashMap;
    87import java.util.LinkedHashMap;
    … …  
    6564        @Override
    6665        public List<RelationMember> sortMembers(List<RelationMember> list) {
    67             Collections.sort(list, (a, b) -> {
     66            list.sort((a, b) -> {
    6867                final int houseNumber = AlphanumComparator.getInstance().compare(
    6968                        a.getMember().get("addr:housenumber"),

  • trunk/src/org/openstreetmap/josm/gui/io/SaveLayersModel.java

    r10611 r10619  
    66import java.io.File;
    77import java.util.ArrayList;
    8 import java.util.Collections;
    98import java.util.List;
    109
    … …  
    7170            layerInfo.add(new SaveLayerInfo(layer));
    7271        }
    73         Collections.sort(layerInfo, (o1, o2) -> o1.compareTo(o2));
     72        layerInfo.sort((o1, o2) -> o1.compareTo(o2));
    7473        fireTableDataChanged();
    7574    }

  • trunk/src/org/openstreetmap/josm/gui/io/UploadSelectionDialog.java

    r10378 r10619  
    202202            if (data == null)
    203203                return;
    204             Collections.sort(
    205                     data,
    206                     new Comparator<OsmPrimitive>() {
    207                         private DefaultNameFormatter formatter = DefaultNameFormatter.getInstance();
    208                         @Override
    209                         public int compare(OsmPrimitive o1, OsmPrimitive o2) {
    210                             int ret = OsmPrimitiveType.from(o1).compareTo(OsmPrimitiveType.from(o2));
    211                             if (ret != 0)
    212                                 return ret;
    213                             return o1.getDisplayName(formatter).compareTo(o1.getDisplayName(formatter));
    214                         }
     204            data.sort(new Comparator<OsmPrimitive>() {
     205                    private DefaultNameFormatter formatter = DefaultNameFormatter.getInstance();
     206                    @Override
     207                    public int compare(OsmPrimitive o1, OsmPrimitive o2) {
     208                        int ret = OsmPrimitiveType.from(o1).compareTo(OsmPrimitiveType.from(o2));
     209                        if (ret != 0)
     210                            return ret;
     211                        return o1.getDisplayName(formatter).compareTo(o1.getDisplayName(formatter));
    215212                    }
     213                }
    216214            );
    217215        }

  • trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java

    r10611 r10619  
    14471447                return;
    14481448            List<Tile> allTiles = allTilesCreate();
    1449             Collections.sort(allTiles, getTileDistanceComparator());
     1449            allTiles.sort(getTileDistanceComparator());
    14501450            for (Tile t : allTiles) {
    14511451                loadTile(t, force);

  • trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

    r10611 r10619  
    646646         */
    647647        final List<Way> sortedWays = new ArrayList<>(ways);
    648         Collections.sort(sortedWays, new OsmPrimitiveComparator(true, false)); // sort by OsmPrimitive#getUniqueId ascending
     648        sortedWays.sort(new OsmPrimitiveComparator(true, false)); // sort by OsmPrimitive#getUniqueId ascending
    649649        Collections.reverse(sortedWays); // sort by OsmPrimitive#getUniqueId descending
    650650        for (Way w : sortedWays) {

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

    r10611 r10619  
    10811081        }
    10821082
    1083         Collections.sort(dateImgLst, (o1, o2) -> o1.getExifTime().compareTo(o2.getExifTime()));
     1083        dateImgLst.sort((o1, o2) -> o1.getExifTime().compareTo(o2.getExifTime()));
    10841084
    10851085        return dateImgLst;

  • trunk/src/org/openstreetmap/josm/gui/layer/gpx/ImportAudioAction.java

    r10611 r10619  
    1111import java.util.Arrays;
    1212import java.util.Collection;
    13 import java.util.Collections;
    1413
    1514import javax.swing.AbstractAction;
    … …  
    278277        /* we must have got at least one waypoint now */
    279278
    280         Collections.sort((ArrayList<WayPoint>) waypoints, (a, b) -> a.time <= b.time ? -1 : 1);
     279        ((ArrayList<WayPoint>) waypoints).sort((a, b) -> a.time <= b.time ? -1 : 1);
    281280
    282281        firstTime = -1.0; /* this time of the first waypoint, not first trackpoint */

  • trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java

    r10611 r10619  
    1919import java.util.ArrayList;
    2020import java.util.Collection;
    21 import java.util.Collections;
    2221import java.util.List;
    2322
    … …  
    234233        if (from instanceof MarkerLayer) {
    235234            data.addAll(((MarkerLayer) from).data);
    236             Collections.sort(data, (o1, o2) -> Double.compare(o1.time, o2.time));
     235            data.sort((o1, o2) -> Double.compare(o1.time, o2.time));
    237236        }
    238237    }

  • trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java

    r10611 r10619  
    12231223
    12241224        protected void sort() {
    1225             Collections.sort(data,
    1226                     (o1, o2) -> {
    1227                         if (o1.isEmpty() && o2.isEmpty())
    1228                             return 0;
    1229                         if (o1.isEmpty()) return 1;
    1230                         if (o2.isEmpty()) return -1;
    1231                         return o1.compareTo(o2);
    1232                     });
     1225            data.sort((o1, o2) -> {
     1226                    if (o1.isEmpty() && o2.isEmpty())
     1227                        return 0;
     1228                    if (o1.isEmpty()) return 1;
     1229                    if (o2.isEmpty()) return -1;
     1230                    return o1.compareTo(o2);
     1231                });
    12331232        }
    12341233

  • trunk/src/org/openstreetmap/josm/gui/preferences/advanced/AdvancedPreference.java

    r10611 r10619  
    274274        readPreferences(tmpPrefs);
    275275        // sorting after modification - first modified, then non-default, then default entries
    276         Collections.sort(allData, customComparator);
     276        allData.sort(customComparator);
    277277        applyFilter();
    278278    }
    … …  
    418418            }
    419419            // allow user to review the changes in table
    420             Collections.sort(allData, customComparator);
     420            allData.sort(customComparator);
    421421            applyFilter();
    422422        }

  • trunk/src/org/openstreetmap/josm/gui/preferences/imagery/CacheContentsPanel.java

    r10611 r10619  
    77import java.awt.event.ActionEvent;
    88import java.util.ArrayList;
    9 import java.util.Collections;
    109import java.util.List;
    1110import java.util.Map;
    … …  
    105104            sortedStats.add(new Pair<>(e.getKey(), e.getValue()[0]));
    106105        }
    107         Collections.sort(sortedStats, (o1, o2) -> -1 * o1.b.compareTo(o2.b));
     106        sortedStats.sort((o1, o2) -> -1 * o1.b.compareTo(o2.b));
    108107        String[][] ret = new String[sortedStats.size()][3];
    109108        int index = 0;

  • trunk/src/org/openstreetmap/josm/gui/preferences/plugin/PluginPreferencesModel.java

    r10611 r10619  
    55import java.util.ArrayList;
    66import java.util.Collection;
    7 import java.util.Collections;
    87import java.util.HashMap;
    98import java.util.HashSet;
    … …  
    147146     */
    148147    protected void sort() {
    149         Collections.sort(
    150                 availablePlugins,
    151                 (o1, o2) -> {
    152                     String n1 = o1.getName() == null ? "" : o1.getName().toLowerCase(Locale.ENGLISH);
    153                     String n2 = o2.getName() == null ? "" : o2.getName().toLowerCase(Locale.ENGLISH);
    154                     return n1.compareTo(n2);
    155                 }
     148        availablePlugins.sort((o1, o2) -> {
     149                String n1 = o1.getName() == null ? "" : o1.getName().toLowerCase(Locale.ENGLISH);
     150                String n2 = o2.getName() == null ? "" : o2.getName().toLowerCase(Locale.ENGLISH);
     151                return n1.compareTo(n2);
     152            }
    156153        );
    157154    }

  • trunk/src/org/openstreetmap/josm/gui/preferences/projection/CodeProjectionChoice.java

    r10378 r10619  
    5959            this.listener = listener;
    6060            data = new ArrayList<>(Projections.getAllProjectionCodes());
    61             Collections.sort(data, new CodeComparator());
     61            data.sort(new CodeComparator());
    6262            filteredData = new ArrayList<>(data);
    6363            build();

  • trunk/src/org/openstreetmap/josm/gui/tagging/TagEditorModel.java

    r10611 r10619  
    88import java.util.ArrayList;
    99import java.util.Collection;
    10 import java.util.Collections;
    1110import java.util.EnumSet;
    1211import java.util.HashMap;
    … …  
    562561     */
    563562    protected void sort() {
    564         Collections.sort(tags, (self, other) -> self.getName().compareTo(other.getName()));
     563        tags.sort((self, other) -> self.getName().compareTo(other.getName()));
    565564    }
    566565

  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetMenu.java

    r10131 r10619  
    1111import java.io.Serializable;
    1212import java.util.ArrayList;
    13 import java.util.Collections;
    1413import java.util.Comparator;
    1514import java.util.List;
    … …  
    140139
    141140    private static void handleMenuItem(JMenu menu, PresetTextComparator comp, List<JMenuItem> sortarray, int lastSeparator) {
    142         Collections.sort(sortarray, comp);
     141        sortarray.sort(comp);
    143142        int pos = 0;
    144143        for (JMenuItem menuItem : sortarray) {

  • trunk/src/org/openstreetmap/josm/io/OsmWriter.java

    r10615 r10619  
    77import java.util.ArrayList;
    88import java.util.Collection;
    9 import java.util.Collections;
    109import java.util.Comparator;
    1110import java.util.List;
    … …  
    110109        List<T> result = new ArrayList<>(primitives.size());
    111110        result.addAll(primitives);
    112         Collections.sort(result, byIdComparator);
     111        result.sort(byIdComparator);
    113112        return result;
    114113    }
    … …  
    269268            }
    270269            List<Entry<String, String>> entries = new ArrayList<>(osm.getKeys().entrySet());
    271             Collections.sort(entries, byKeyComparator);
     270            entries.sort(byKeyComparator);
    272271            for (Entry<String, String> e : entries) {
    273272                out.println("    <tag k='"+ XmlWriter.encode(e.getKey()) +

  • trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java

    r10616 r10619  
    746746            // lower the value of "stage" the earlier the plugin should be loaded.
    747747            //
    748             Collections.sort(
    749                     toLoad,
    750                     (o1, o2) -> {
    751                         if (o1.stage < o2.stage) return -1;
    752                         if (o1.stage == o2.stage) return 0;
    753                         return 1;
    754                     }
     748            toLoad.sort((o1, o2) -> {
     749                    if (o1.stage < o2.stage) return -1;
     750                    if (o1.stage == o2.stage) return 0;
     751                    return 1;
     752                }
    755753            );
    756754            if (toLoad.isEmpty())
Note: See TracChangeset for help on using the changeset viewer.