Changeset 16468 in josm


Ignore:
Timestamp:
2020-05-20T22:51:46+02:00 (2 weeks ago)
Author:
simon04
Message:

Java 8: use Collection.removeIf

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

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/UndoRedoHandler.java

    r15586 r16468  
    33
    44import java.util.EventObject;
    5 import java.util.Iterator;
    65import java.util.LinkedList;
    76import java.util.List;
     
    452451            return;
    453452        boolean changed = false;
    454         for (Iterator<Command> it = commands.iterator(); it.hasNext();) {
    455             if (it.next().getAffectedDataSet() == dataSet) {
    456                 it.remove();
    457                 changed = true;
    458             }
    459         }
    460         for (Iterator<Command> it = redoCommands.iterator(); it.hasNext();) {
    461             if (it.next().getAffectedDataSet() == dataSet) {
    462                 it.remove();
    463                 changed = true;
    464             }
    465         }
     453        changed |= commands.removeIf(c -> c.getAffectedDataSet() == dataSet);
     454        changed |= redoCommands.removeIf(c -> c.getAffectedDataSet() == dataSet);
    466455        if (changed) {
    467456            fireEvent(new CommandQueueCleanedEvent(this, dataSet));
  • trunk/src/org/openstreetmap/josm/data/osm/DataSet.java

    r16445 r16468  
    1010import java.util.HashMap;
    1111import java.util.HashSet;
    12 import java.util.Iterator;
    1312import java.util.LinkedHashSet;
    1413import java.util.LinkedList;
     
    820819            for (Relation relation : getRelations()) {
    821820                List<RelationMember> members = relation.getMembers();
    822 
    823                 Iterator<RelationMember> it = members.iterator();
    824                 boolean removed = false;
    825                 while (it.hasNext()) {
    826                     RelationMember member = it.next();
    827                     if (member.getMember().equals(primitive)) {
    828                         it.remove();
    829                         removed = true;
    830                     }
    831                 }
    832 
     821                boolean removed = members.removeIf(member -> member.getMember().equals(primitive));
    833822                if (removed) {
    834823                    relation.setMembers(members);
  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/MapRendererFactory.java

    r16445 r16468  
    99import java.util.ArrayList;
    1010import java.util.Collections;
    11 import java.util.Iterator;
    1211import java.util.List;
    1312
     
    239238        if (renderer == null) return;
    240239        if (!isRegistered(renderer)) return;
    241         Iterator<Descriptor> it = descriptors.iterator();
    242         while (it.hasNext()) {
    243             Descriptor d = it.next();
    244             if (d.getRenderer().equals(renderer)) {
    245                 it.remove();
    246                 break;
    247             }
    248         }
     240        descriptors.removeIf(d -> d.getRenderer().equals(renderer));
    249241        if (activeRenderer != null && activeRenderer.equals(renderer)) {
    250242            activateDefault();
  • trunk/src/org/openstreetmap/josm/data/preferences/sources/MapPaintPrefHelper.java

    r15719 r16468  
    7474
    7575        // XML style is not bundled anymore
    76         list.stream()
    77                 .filter(se -> "resource://styles/standard/elemstyles.xml".equals(se.url))
    78                 .findFirst()
    79                 .ifPresent(list::remove);
     76        list.removeIf(se -> "resource://styles/standard/elemstyles.xml".equals(se.url));
    8077
    8178        return changed;
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/RelationDialogManager.java

    r16438 r16468  
    108108    public void updateContext(OsmDataLayer layer, Relation relation, RelationEditor editor) {
    109109        // lookup the entry for editor and remove it
    110         for (Iterator<Entry<DialogContext, RelationEditor>> it = openDialogs.entrySet().iterator(); it.hasNext();) {
    111             Entry<DialogContext, RelationEditor> entry = it.next();
    112             if (Objects.equals(entry.getValue(), editor)) {
    113                 it.remove();
    114                 break;
    115             }
    116         }
     110        openDialogs.values().removeIf(i -> Objects.equals(i, editor));
    117111        // don't add a window listener. Editor is already known to the relation dialog manager
    118112        openDialogs.put(new DialogContext(layer, relation), editor);
     
    199193        if (w instanceof RelationEditor) {
    200194            RelationEditor editor = (RelationEditor) w;
    201             for (Iterator<Entry<DialogContext, RelationEditor>> it = openDialogs.entrySet().iterator(); it.hasNext();) {
    202                 if (editor.equals(it.next().getValue())) {
    203                     it.remove();
    204                     break;
    205                 }
    206             }
     195            openDialogs.values().removeIf(i -> Objects.equals(i, editor));
    207196        }
    208197    }
  • trunk/src/org/openstreetmap/josm/gui/io/UploadDialog.java

    r15802 r16468  
    2121import java.util.Collections;
    2222import java.util.HashMap;
    23 import java.util.Iterator;
    2423import java.util.List;
    2524import java.util.Locale;
     
    323322
    324323        // remove empty values
    325         final Iterator<String> it = tags.keySet().iterator();
    326         while (it.hasNext()) {
    327             final String v = tags.get(it.next());
    328             if (v == null || v.isEmpty()) {
    329                 it.remove();
    330             }
    331         }
     324        tags.keySet().removeIf(key -> {
     325            final String v = tags.get(key);
     326            return v == null || v.isEmpty();
     327        });
    332328
    333329        // ignore source/comment to keep current values from models ?
  • trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java

    r16438 r16468  
    14301430                }
    14311431                readFile();
    1432                 for (Iterator<ExtendedSourceEntry> it = sources.iterator(); it.hasNext();) {
    1433                     if ("xml".equals(it.next().styleType)) {
    1434                         Logging.debug("Removing XML source entry");
    1435                         it.remove();
    1436                     }
     1432                if (sources.removeIf(extendedSourceEntry -> "xml".equals(extendedSourceEntry.styleType))) {
     1433                    Logging.debug("Removing XML source entry");
    14371434                }
    14381435            } catch (IOException e) {
  • trunk/src/org/openstreetmap/josm/gui/tagging/TagEditorModel.java

    r16438 r16468  
    1111import java.util.Comparator;
    1212import java.util.EnumSet;
    13 import java.util.Iterator;
    1413import java.util.List;
    1514import java.util.Map;
     
    321320        if (name == null)
    322321            return;
    323         Iterator<TagModel> it = tags.iterator();
    324         boolean changed = false;
    325         while (it.hasNext()) {
    326             TagModel tm = it.next();
    327             if (tm.getName().equals(name)) {
    328                 changed = true;
    329                 it.remove();
    330             }
    331         }
     322        boolean changed = tags.removeIf(tm -> tm.getName().equals(name));
    332323        if (changed) {
    333324            fireTableDataChanged();
Note: See TracChangeset for help on using the changeset viewer.