Ticket #13785: use-of-streams.patch

File use-of-streams.patch, 5.8 KB (added by alno, 9 years ago)

  • org/openstreetmap/josm/data/osm/Changeset.java 

    # This patch file was generated by NetBeans IDE
# Following Index: paths are relative to: C:\Users\alexandre.nouvel\Documents\NetBeansProjects\trunk\src
# This patch can be applied using context Tools: Patch action on respective folder.
# It uses platform neutral UTF-8 encoding and \n newlines.
# Above lines and this line are ignored by the patching process.
     
    207207    @Override
    208208    public void setKeys(Map<String, String> keys) {
    209209        CheckParameterUtil.ensureParameterNotNull(keys, "keys");
    210         for (String value : keys.values()) {
    211             if (value != null && value.length() > MAX_CHANGESET_TAG_LENGTH) {
     210        keys.values().stream()
     211                .filter(value -> (value != null && value.length() > MAX_CHANGESET_TAG_LENGTH))
     212                .findFirst()
     213                .ifPresent(value -> {
    212214                throw new IllegalArgumentException("Changeset tag value is too long: "+value);
    213             }
    214         }
     215        });
    215216        this.tags = keys;
    216217    }
    217218

  • org/openstreetmap/josm/data/osm/ChangesetCache.java

     
    99import java.util.Map;
    1010import java.util.Set;
    1111import java.util.concurrent.CopyOnWriteArrayList;
     12import java.util.stream.Collectors;
    1213
    1314import org.openstreetmap.josm.Main;
    1415import org.openstreetmap.josm.data.Preferences.PreferenceChangeEvent;
    … …  
    182183     * @return The list of open changesets
    183184     */
    184185    public List<Changeset> getOpenChangesets() {
    185         List<Changeset> ret = new ArrayList<>();
    186         for (Changeset cs: cache.values()) {
    187             if (cs.isOpen()) {
    188                 ret.add(cs);
    189             }
    190         }
     186        List<Changeset> ret = cache.values().stream()
     187                .filter(cs -> cs.isOpen())
     188                .collect(Collectors.toList());
    191189        return ret;
    192190    }
    193191

  • org/openstreetmap/josm/data/osm/ChangesetDataSet.java

     
    77import java.util.Map;
    88import java.util.Map.Entry;
    99import java.util.Set;
     10import java.util.stream.Collectors;
    1011
    1112import org.openstreetmap.josm.data.osm.history.HistoryOsmPrimitive;
    1213import org.openstreetmap.josm.tools.CheckParameterUtil;
    … …  
    116117     */
    117118    public Set<HistoryOsmPrimitive> getPrimitivesByModificationType(ChangesetModificationType cmt) {
    118119        CheckParameterUtil.ensureParameterNotNull(cmt, "cmt");
    119         Set<HistoryOsmPrimitive> ret = new HashSet<>();
    120         for (Entry<PrimitiveId, ChangesetModificationType> entry: modificationTypes.entrySet()) {
    121             if (entry.getValue().equals(cmt)) {
    122                 ret.add(primitives.get(entry.getKey()));
    123             }
    124         }
     120        Set<HistoryOsmPrimitive> ret = modificationTypes.entrySet().stream()
     121                .filter(entry -> entry.getValue().equals(cmt))
     122                .map(entry -> primitives.get(entry.getKey()))
     123                .collect(Collectors.toSet());
    125124        return ret;
    126125    }
    127126

  • org/openstreetmap/josm/data/osm/DataSet.java

     
    2121import java.util.concurrent.locks.ReadWriteLock;
    2222import java.util.concurrent.locks.ReentrantReadWriteLock;
    2323import java.util.function.Predicate;
     24import java.util.stream.Collectors;
    2425
    2526import org.openstreetmap.josm.Main;
    2627import org.openstreetmap.josm.data.Bounds;
    … …  
    421422        lock.readLock().lock();
    422423        try {
    423424            // QuadBuckets might be useful here (don't forget to do reindexing after some of rm is changed)
    424             List<Relation> result = new ArrayList<>();
    425             for (Relation r: relations) {
    426                 if (r.getBBox().intersects(bbox)) {
    427                     result.add(r);
    428                 }
    429             }
     425            List<Relation> result = relations.stream()
     426                    .filter(r -> r.getBBox().intersects(bbox))
     427                    .collect(Collectors.toList());
    430428            return result;
    431429        } finally {
    432430            lock.readLock().unlock();

  • org/openstreetmap/josm/gui/conflict/pair/ListMergeModel.java

     
    225225    protected void fireFrozenChanged(boolean oldValue, boolean newValue) {
    226226        synchronized (listeners) {
    227227            PropertyChangeEvent evt = new PropertyChangeEvent(this, FROZEN_PROP, oldValue, newValue);
    228             for (PropertyChangeListener listener: listeners) {
     228            listeners.stream().forEach((listener) -> {
    229229                listener.propertyChange(evt);
     230            });
    230231            }
    231232        }
    232     }
    233233
    234234    public final void setFrozen(boolean isFrozen) {
    235235        boolean oldValue = this.isFrozen;
    … …  
    677677
    678678            if (row >= getEntries().size()) return false;
    679679            T e1 = getEntries().get(row);
    680             for (T e2: getOppositeEntries()) {
    681                 if (isEqualEntry(e1, e2)) return true;
     680            return (getOppositeEntries().stream().anyMatch((e2) -> (isEqualEntry(e1, e2))));
    682681            }
    683             return false;
    684         }
    685682
    686683        protected List<T> getEntries() {
    687684            return entries.get(role);