Changeset 11339 in josm


Ignore:
Timestamp:
2016-11-28T21:18:55+01:00 (7 years ago)
Author:
simon04
Message:

Loop can be replaced with Collection.removeIf()

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/Main.java

    r11284 r11339  
    10921092        if (listener == null) return;
    10931093        synchronized (Main.class) {
    1094             Iterator<WeakReference<ProjectionChangeListener>> it = listeners.iterator();
    1095             while (it.hasNext()) {
    1096                 WeakReference<ProjectionChangeListener> wr = it.next();
    1097                 // remove the listener - and any other listener which got garbage
    1098                 // collected in the meantime
    1099                 if (wr.get() == null || wr.get() == listener) {
    1100                     it.remove();
    1101                 }
    1102             }
     1094            // remove the listener - and any other listener which got garbage
     1095            // collected in the meantime
     1096            listeners.removeIf(wr -> wr.get() == null || wr.get() == listener);
    11031097        }
    11041098    }
    … …  
    11561150        if (listener == null) return;
    11571151        synchronized (Main.class) {
    1158             Iterator<WeakReference<WindowSwitchListener>> it = windowSwitchListeners.iterator();
    1159             while (it.hasNext()) {
    1160                 WeakReference<WindowSwitchListener> wr = it.next();
    1161                 // remove the listener - and any other listener which got garbage
    1162                 // collected in the meantime
    1163                 if (wr.get() == null || wr.get() == listener) {
    1164                     it.remove();
    1165                 }
    1166             }
     1152            // remove the listener - and any other listener which got garbage
     1153            // collected in the meantime
     1154            windowSwitchListeners.removeIf(wr -> wr.get() == null || wr.get() == listener);
    11671155            if (windowSwitchListeners.isEmpty()) {
    11681156                MasterWindowListener.teardown();

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

    r10383 r11339  
    88import java.util.ArrayList;
    99import java.util.Collection;
    10 import java.util.Iterator;
    1110import java.util.List;
    1211
    … …  
    1514import org.openstreetmap.josm.Main;
    1615import org.openstreetmap.josm.data.notes.Note;
     16import org.openstreetmap.josm.data.osm.AbstractPrimitive;
    1717import org.openstreetmap.josm.data.osm.DataSet;
    1818import org.openstreetmap.josm.data.osm.OsmPrimitive;
    … …  
    109109        // filter out new primitives which are not yet uploaded to the server
    110110        //
    111         Iterator<OsmPrimitive> it = primitivesToShow.iterator();
    112         while (it.hasNext()) {
    113             if (it.next().isNew()) {
    114                 it.remove();
    115             }
    116         }
     111        primitivesToShow.removeIf(AbstractPrimitive::isNew);
    117112
    118113        if (primitivesToShow.isEmpty() && noteToShow == null) {

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

    r10467 r11339  
    1010import java.util.ArrayList;
    1111import java.util.Collection;
    12 import java.util.Iterator;
    1312import java.util.LinkedList;
    1413import java.util.List;
    … …  
    156155        for (int i = 1; i < selectedNodes.size(); i++) {
    157156            List<OsmPrimitive> ref = selectedNodes.get(i).getReferrers();
    158             for (Iterator<Way> it = result.iterator(); it.hasNext();) {
    159                 if (!ref.contains(it.next())) {
    160                     it.remove();
    161                 }
    162             }
     157            result.removeIf(way -> !ref.contains(way));
    163158        }
    164159
    165160        // Remove broken ways
    166         for (Iterator<Way> it = result.iterator(); it.hasNext();) {
    167             if (it.next().getNodesCount() <= 2) {
    168                 it.remove();
    169             }
    170         }
     161        result.removeIf(way -> way.getNodesCount() <= 2);
    171162
    172163        if (selectedWays.isEmpty())
    … …  
    174165        else {
    175166            // Return only selected ways
    176             for (Iterator<Way> it = result.iterator(); it.hasNext();) {
    177                 if (!selectedWays.contains(it.next())) {
    178                     it.remove();
    179                 }
    180             }
     167            result.removeIf(way -> !selectedWays.contains(way));
    181168            return result;
    182169        }

  • trunk/src/org/openstreetmap/josm/command/DeleteCommand.java

    r11240 r11339  
    1414import java.util.HashMap;
    1515import java.util.HashSet;
    16 import java.util.Iterator;
    1716import java.util.LinkedList;
    1817import java.util.List;
    … …  
    439438        if (!silent) {
    440439            Set<RelationToChildReference> references = RelationToChildReference.getRelationToChildReferences(primitivesToDelete);
    441             Iterator<RelationToChildReference> it = references.iterator();
    442             while (it.hasNext()) {
    443                 RelationToChildReference ref = it.next();
    444                 if (ref.getParent().isDeleted()) {
    445                     it.remove();
    446                 }
    447             }
     440            references.removeIf(ref -> ref.getParent().isDeleted());
    448441            if (!references.isEmpty()) {
    449442                DeleteFromRelationConfirmationDialog dialog = DeleteFromRelationConfirmationDialog.getInstance();

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

    r10608 r11339  
    10881088            }
    10891089        }
    1090         Iterator<Map.Entry<String, Setting<?>>> it = tmpPref.settingsMap.entrySet().iterator();
    1091         while (it.hasNext()) {
    1092             Map.Entry<String, Setting<?>> e = it.next();
    1093             if (e.getValue().getValue() == null) {
    1094                 it.remove();
    1095             }
    1096         }
     1090        tmpPref.settingsMap.entrySet().removeIf(e -> e.getValue().getValue() == null);
    10971091
    10981092        for (Map.Entry<String, Setting<?>> e: tmpPref.settingsMap.entrySet()) {

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

    r11097 r11339  
    291291    public void removeByKey(String key) {
    292292        if (key != null) {
    293             Iterator<Tag> it = tags.keySet().iterator();
    294             while (it.hasNext()) {
    295                 if (it.next().matchesKey(key)) {
    296                     it.remove();
    297                 }
    298             }
     293            tags.keySet().removeIf(tag -> tag.matchesKey(key));
    299294        }
    300295    }

  • trunk/src/org/openstreetmap/josm/data/validation/tests/Addresses.java

    r11129 r11339  
    99import java.util.HashMap;
    1010import java.util.HashSet;
    11 import java.util.Iterator;
    1211import java.util.List;
    1312import java.util.Locale;
    … …  
    5958    protected List<Relation> getAndCheckAssociatedStreets(OsmPrimitive p) {
    6059        List<Relation> list = OsmPrimitive.getFilteredList(p.getReferrers(), Relation.class);
    61         for (Iterator<Relation> it = list.iterator(); it.hasNext();) {
    62             Relation r = it.next();
    63             if (!r.hasTag("type", ASSOCIATED_STREET)) {
    64                 it.remove();
    65             }
    66         }
     60        list.removeIf(r -> !r.hasTag("type", ASSOCIATED_STREET));
    6761        if (list.size() > 1) {
    6862            Severity level;

  • trunk/src/org/openstreetmap/josm/data/validation/tests/DuplicateNode.java

    r11268 r11339  
    2121import org.openstreetmap.josm.command.Command;
    2222import org.openstreetmap.josm.data.coor.LatLon;
     23import org.openstreetmap.josm.data.osm.AbstractPrimitive;
    2324import org.openstreetmap.josm.data.osm.Hash;
    2425import org.openstreetmap.josm.data.osm.Node;
    … …  
    315316
    316317        // Filter nodes that have already been deleted (see #5764 and #5773)
    317         for (Iterator<Node> it = nodes.iterator(); it.hasNext();) {
    318             if (it.next().isDeleted()) {
    319                 it.remove();
    320             }
    321         }
     318        nodes.removeIf(AbstractPrimitive::isDeleted);
    322319
    323320        // Merge only if at least 2 nodes remain

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

    r9981 r11339  
    6464    public void setTagsToIgnore(SearchCompiler.Match tagsToIgnore) {
    6565        this.tagsToIgnore = tagsToIgnore;
    66         final Iterator<Tag> it = recentTags.keySet().iterator();
    67         while (it.hasNext()) {
    68             if (tagsToIgnore.match(it.next())) {
    69                 it.remove();
    70             }
    71         }
     66        recentTags.keySet().removeIf(tagsToIgnore::match);
    7267    }
    7368

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

    r11304 r11339  
    3333import java.util.Comparator;
    3434import java.util.HashMap;
    35 import java.util.Iterator;
    3635import java.util.List;
    3736import java.util.Map;
    … …  
    687686
    688687            // remove the object's tag keys from the list
    689             Iterator<AutoCompletionListItem> iter = keyList.iterator();
    690             while (iter.hasNext()) {
    691                 AutoCompletionListItem item = iter.next();
    692                 if (containsDataKey(item.getValue())) {
    693                     iter.remove();
    694                 }
    695             }
     688            keyList.removeIf(item -> containsDataKey(item.getValue()));
    696689
    697690            keyList.sort(defaultACItemComparator);

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

    r10777 r11339  
    99import java.util.EnumSet;
    1010import java.util.HashSet;
    11 import java.util.Iterator;
    1211import java.util.List;
    1312import java.util.Set;
    … …  
    349348        if (primitives == null)
    350349            return;
    351         Iterator<RelationMember> it = members.iterator();
    352         while (it.hasNext()) {
    353             RelationMember member = it.next();
    354             if (primitives.contains(member.getMember())) {
    355                 it.remove();
    356             }
    357         }
     350        members.removeIf(member -> primitives.contains(member.getMember()));
    358351        fireTableDataChanged();
    359352    }

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

    r11198 r11339  
    2020import java.util.Collection;
    2121import java.util.Collections;
    22 import java.util.Iterator;
    2322import java.util.LinkedList;
    2423import java.util.List;
    … …  
    430429                // select plugins which actually have to be updated
    431430                //
    432                 Iterator<PluginInformation> it = toUpdate.iterator();
    433                 while (it.hasNext()) {
    434                     PluginInformation pi = it.next();
    435                     if (!pi.isUpdateRequired()) {
    436                         it.remove();
    437                     }
    438                 }
     431                toUpdate.removeIf(pi -> !pi.isUpdateRequired());
    439432                if (toUpdate.isEmpty()) {
    440433                    alertNothingToUpdate();

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

    r11288 r11339  
    547547            // Build list of plugins to download
    548548            Set<PluginInformation> toDownload = new HashSet<>(pluginInfoDownloadTask.getAvailablePlugins());
    549             for (Iterator<PluginInformation> it = toDownload.iterator(); it.hasNext();) {
    550                 PluginInformation info = it.next();
    551                 if (!missingRequiredPlugin.contains(info.getName())) {
    552                     it.remove();
    553                 }
    554             }
     549            toDownload.removeIf(info -> !missingRequiredPlugin.contains(info.getName()));
    555550            // Check if something has still to be downloaded
    556551            if (!toDownload.isEmpty()) {
Note: See TracChangeset for help on using the changeset viewer.