Changeset 10647 in josm for trunk


Ignore:
Timestamp:
2016-07-26T02:17:04+02:00 (8 years ago)
Author:
Don-vip
Message:

see #11390 - use functional comparators

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

Legend:

Unmodified
Added
Removed

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

    r10619 r10647  
    44import java.util.ArrayList;
    55import java.util.Collection;
     6import java.util.Comparator;
    67import java.util.HashMap;
    78import java.util.HashSet;
    … …  
    331332            }
    332333            List<Relation> ret = new ArrayList<>(relations);
    333             ret.sort((o1, o2) -> Integer.compare(uploadOrder.indexOf(o1), uploadOrder.indexOf(o2)));
     334            ret.sort(Comparator.comparingInt(uploadOrder::indexOf));
    334335            return ret;
    335336        }

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

    r10619 r10647  
    44import java.text.MessageFormat;
    55import java.util.ArrayList;
     6import java.util.Comparator;
    67import java.util.Date;
    78import java.util.List;
    … …  
    6869    public History sortAscending() {
    6970        List<HistoryOsmPrimitive> copy = new ArrayList<>(versions);
    70         copy.sort((o1, o2) -> o1.compareTo(o2));
     71        copy.sort(Comparator.naturalOrder());
    7172        return new History(id, type, copy);
    7273    }
    … …  
    7879    public History sortDescending() {
    7980        List<HistoryOsmPrimitive> copy = new ArrayList<>(versions);
    80         copy.sort((o1, o2) -> o2.compareTo(o1));
     81        copy.sort(Comparator.reverseOrder());
    8182        return new History(id, type, copy);
    8283    }

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

    r10619 r10647  
    66import java.util.ArrayList;
    77import java.util.Collection;
     8import java.util.Comparator;
    89import java.util.HashSet;
    910import java.util.List;
    … …  
    150151
    151152    protected void sort() {
    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             }
    157         );
     153        data.sort(Comparator.comparingInt(Changeset::getId).reversed());
    158154    }
    159155

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

    r10619 r10647  
    44import java.util.ArrayList;
    55import java.util.Collection;
     6import java.util.Comparator;
    67import java.util.HashSet;
    78import java.util.List;
    … …  
    139140
    140141    protected void sort() {
    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             }
    146         );
     142        data.sort(Comparator.comparingInt(Changeset::getId).reversed());
    147143    }
    148144

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

    r10619 r10647  
    66import java.io.File;
    77import java.util.ArrayList;
     8import java.util.Comparator;
    89import java.util.List;
    910
    … …  
    7071            layerInfo.add(new SaveLayerInfo(layer));
    7172        }
    72         layerInfo.sort((o1, o2) -> o1.compareTo(o2));
     73        layerInfo.sort(Comparator.naturalOrder());
    7374        fireTableDataChanged();
    7475    }

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

    r10632 r10647  
    3030import java.util.Collection;
    3131import java.util.Collections;
     32import java.util.Comparator;
    3233import java.util.Date;
    3334import java.util.Dictionary;
    … …  
    10821083        }
    10831084
    1084         dateImgLst.sort((o1, o2) -> o1.getExifTime().compareTo(o2.getExifTime()));
     1085        dateImgLst.sort(Comparator.comparing(ImageEntry::getExifTime));
    10851086
    10861087        return dateImgLst;

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

    r10619 r10647  
    1111import java.util.Arrays;
    1212import java.util.Collection;
     13import java.util.Comparator;
    1314
    1415import javax.swing.AbstractAction;
    … …  
    8788            // long as they don't overlap, that's fine)
    8889            if (sel.length > 1) {
    89                 Arrays.sort(sel, (a, b) -> a.lastModified() <= b.lastModified() ? -1 : 1);
     90                Arrays.sort(sel, Comparator.comparingLong(File::lastModified));
    9091            }
    9192            StringBuilder names = new StringBuilder();
    … …  
    277278        /* we must have got at least one waypoint now */
    278279
    279         ((ArrayList<WayPoint>) waypoints).sort((a, b) -> a.time <= b.time ? -1 : 1);
     280        ((ArrayList<WayPoint>) waypoints).sort(Comparator.comparingDouble(o -> o.time));
    280281
    281282        firstTime = -1.0; /* this time of the first waypoint, not first trackpoint */

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

    r10619 r10647  
    1919import java.util.ArrayList;
    2020import java.util.Collection;
     21import java.util.Comparator;
    2122import java.util.List;
    2223
    … …  
    233234        if (from instanceof MarkerLayer) {
    234235            data.addAll(((MarkerLayer) from).data);
    235             data.sort((o1, o2) -> Double.compare(o1.time, o2.time));
     236            data.sort(Comparator.comparingDouble(o -> o.time));
    236237        }
    237238    }

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

    r10619 r10647  
    77import java.awt.event.ActionEvent;
    88import java.util.ArrayList;
     9import java.util.Comparator;
    910import java.util.List;
    1011import java.util.Map;
    … …  
    104105            sortedStats.add(new Pair<>(e.getKey(), e.getValue()[0]));
    105106        }
    106         sortedStats.sort((o1, o2) -> -1 * o1.b.compareTo(o2.b));
     107        sortedStats.sort(Comparator.comparing(o -> o.b, Comparator.reverseOrder()));
    107108        String[][] ret = new String[sortedStats.size()][3];
    108109        int index = 0;

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

    r10619 r10647  
    55import java.util.ArrayList;
    66import java.util.Collection;
     7import java.util.Comparator;
    78import java.util.HashMap;
    89import java.util.HashSet;
    … …  
    146147     */
    147148    protected void sort() {
    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             }
    153         );
     149        availablePlugins.sort(Comparator.comparing(
     150                o -> o.getName() == null ? "" : o.getName().toLowerCase(Locale.ENGLISH)));
    154151    }
    155152

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

    r10619 r10647  
    88import java.util.ArrayList;
    99import java.util.Collection;
     10import java.util.Comparator;
    1011import java.util.EnumSet;
    1112import java.util.HashMap;
    … …  
    561562     */
    562563    protected void sort() {
    563         tags.sort((self, other) -> self.getName().compareTo(other.getName()));
     564        tags.sort(Comparator.comparing(TagModel::getName));
    564565    }
    565566

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

    r10627 r10647  
    2424import java.util.Collection;
    2525import java.util.Collections;
     26import java.util.Comparator;
    2627import java.util.HashMap;
    2728import java.util.HashSet;
    … …  
    745746            // lower the value of "stage" the earlier the plugin should be loaded.
    746747            //
    747             toLoad.sort((o1, o2) -> {
    748                     if (o1.stage < o2.stage) return -1;
    749                     if (o1.stage == o2.stage) return 0;
    750                     return 1;
    751                 }
    752             );
     748            toLoad.sort(Comparator.comparingInt(o -> o.stage));
    753749            if (toLoad.isEmpty())
    754750                return;

  • trunk/src/org/openstreetmap/josm/tools/I18n.java

    r10627 r10647  
    1616import java.util.Arrays;
    1717import java.util.Collection;
     18import java.util.Comparator;
    1819import java.util.HashMap;
    1920import java.util.Locale;
    … …  
    381382        Locale[] l = new Locale[v.size()];
    382383        l = v.toArray(l);
    383         Arrays.sort(l, (o1, o2) -> o1.toString().compareTo(o2.toString()));
     384        Arrays.sort(l, Comparator.comparing(Locale::toString));
    384385        return l;
    385386    }
Note: See TracChangeset for help on using the changeset viewer.