Changeset 10601 in josm for trunk


Ignore:
Timestamp:
2016-07-23T04:10:39+02:00 (8 years ago)
Author:
Don-vip
Message:

see #11390 - sonar - squid:S1604 - Java 8: Anonymous inner classes containing only one method should become lambdas

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

Legend:

Unmodified
Added
Removed

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

    r10580 r10601  
    596596    public Main() {
    597597        main = this;
    598         mainPanel.addMapFrameListener(new MapFrameListener() {
    599             @Override
    600             public void mapFrameInitialized(MapFrame oldFrame, MapFrame newFrame) {
    601                 redoUndoListener.commandChanged(0, 0);
    602             }
    603         });
     598        mainPanel.addMapFrameListener((o, n) -> redoUndoListener.commandChanged(0, 0));
    604599    }
    605600
    … …  
    697692
    698693        // hooks for the jmapviewer component
    699         FeatureAdapter.registerBrowserAdapter(new FeatureAdapter.BrowserAdapter() {
    700             @Override
    701             public void openLink(String url) {
    702                 OpenBrowser.displayUrl(url);
    703             }
    704         });
     694        FeatureAdapter.registerBrowserAdapter(OpenBrowser::displayUrl);
    705695        FeatureAdapter.registerTranslationAdapter(I18n.getTranslationAdapter());
    706         FeatureAdapter.registerLoggingAdapter(new FeatureAdapter.LoggingAdapter() {
    707             @Override
    708             public Logger getLogger(String name) {
     696        FeatureAdapter.registerLoggingAdapter(name -> {
    709697                Logger logger = Logger.getAnonymousLogger();
    710698                logger.setUseParentHandlers(false);
    … …  
    740728                }
    741729                return logger;
    742             }
    743         });
     730            });
    744731
    745732        new InitializationTask(tr("Updating user interface")) {
    … …  
    991978    public static final JPanel panel = mainPanel;
    992979
    993     private final CommandQueueListener redoUndoListener = new CommandQueueListener() {
    994         @Override
    995         public void commandChanged(final int queueSize, final int redoSize) {
     980    private final CommandQueueListener redoUndoListener = (queueSize, redoSize) -> {
    996981            menu.undo.setEnabled(queueSize > 0);
    997982            menu.redo.setEnabled(redoSize > 0);
    998         }
    999     };
     983        };
    1000984
    1001985    /**

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

    r10463 r10601  
    3232import org.openstreetmap.josm.tools.GBC;
    3333import org.openstreetmap.josm.tools.ImageProvider;
    34 import org.openstreetmap.josm.tools.ImageProvider.ImageResourceCallback;
    3534import org.openstreetmap.josm.tools.ImageResource;
    3635
    … …  
    5756        String icon = info.getIcon();
    5857        if (icon != null) {
    59             new ImageProvider(icon).setOptional(true).getInBackground(new ImageResourceCallback() {
    60                         @Override
    61                         public void finished(final ImageResource result) {
    62                             if (result != null) {
    63                                 GuiHelper.runInEDT(new Runnable() {
    64                                     @Override
    65                                     public void run() {
    66                                         result.attachImageIcon(AddImageryLayerAction.this);
    67                                     }
    68                                 });
    69                             }
    70                         }
    71                     });
     58            new ImageProvider(icon).setOptional(true).getInBackground((ImageResource result) -> {
     59                if (result != null) {
     60                    GuiHelper.runInEDT(() -> result.attachImageIcon(AddImageryLayerAction.this));
     61                }
     62            });
    7263        }
    7364    }

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

    r10453 r10601  
    1717
    1818import javax.swing.JOptionPane;
    19 import javax.swing.event.ListSelectionEvent;
    2019import javax.swing.event.ListSelectionListener;
    21 import javax.swing.event.TreeSelectionEvent;
    2220import javax.swing.event.TreeSelectionListener;
    2321
    … …  
    410408        MapFrameAdapter() {
    411409            if ("conflict".equals(mode)) {
    412                 conflictSelectionListener = new ListSelectionListener() {
    413                     @Override
    414                     public void valueChanged(ListSelectionEvent e) {
    415                         updateEnabledState();
    416                     }
    417                 };
     410                conflictSelectionListener = e -> updateEnabledState();
    418411            } else if ("problem".equals(mode)) {
    419                 validatorSelectionListener = new TreeSelectionListener() {
    420                     @Override
    421                     public void valueChanged(TreeSelectionEvent e) {
    422                         updateEnabledState();
    423                     }
    424                 };
     412                validatorSelectionListener = e -> updateEnabledState();
    425413            }
    426414        }

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

    r10212 r10601  
    9797        @Override
    9898        protected void finish() {
    99             SwingUtilities.invokeLater(
    100                     new Runnable() {
    101                         @Override
    102                         public void run() {
     99            SwingUtilities.invokeLater(() -> {
    103100                            if (lastException != null) {
    104101                                ExceptionDialogUtil.explainException(lastException);
    … …  
    108105                                onPostDownloadOpenChangesets();
    109106                            }
    110                         }
    111                     }
    112             );
     107                        });
    113108        }
    114109

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

    r10548 r10601  
    233233        Main.main.undoRedo.add(combineResult.b);
    234234        if (selectedWay != null) {
    235             Runnable guiTask = new Runnable() {
    236                 @Override
    237                 public void run() {
    238                     ds.setSelected(selectedWay);
    239                 }
    240             };
    241             GuiHelper.runInEDT(guiTask);
     235            GuiHelper.runInEDT(() -> ds.setSelected(selectedWay));
    242236        }
    243237    }

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

    r10548 r10601  
    104104
    105105            // to avoid EDT violations
    106             SwingUtilities.invokeLater(new Runnable() {
    107                 @Override
    108                 public void run() {
     106            SwingUtilities.invokeLater(() -> {
    109107                    Main.main.undoRedo.add(command);
    110108
    … …  
    113111                    // (Yes, we are already in event dispatch thread. But DatasetEventManager
    114112                    // uses 'SwingUtilities.invokeLater' to fire events so we have to do the same.)
    115                     SwingUtilities.invokeLater(new Runnable() {
    116                         @Override
    117                         public void run() {
     113                    SwingUtilities.invokeLater(() -> {
    118114                            Main.map.relationListDialog.selectRelation(relation);
    119115                            if (Main.pref.getBoolean("multipoly.show-relation-editor", false)) {
    120116                                //Open relation edit window, if set up in preferences
    121117                                RelationEditor editor = RelationEditor.getEditor(Main.getLayerManager().getEditLayer(), relation, null);
    122 
    123118                                editor.setModal(true);
    124119                                editor.setVisible(true);
    … …  
    126121                                Main.getLayerManager().getEditLayer().setRecentRelation(relation);
    127122                            }
    128                         }
    129123                    });
    130                 }
    131124            });
    132125        }
    … …  
    296289        if (error != null) {
    297290            if (showNotif) {
    298                 GuiHelper.runInEDT(new Runnable() {
    299                     @Override
    300                     public void run() {
     291                GuiHelper.runInEDT(
    301292                        new Notification(error)
    302293                        .setIcon(JOptionPane.INFORMATION_MESSAGE)
    303                         .show();
    304                     }
    305                 });
     294                        ::show);
    306295            }
    307296            return null;

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

    r10446 r10601  
    5353                new DownloadPrimitivesWithReferrersTask(newLayer, ids, downloadReferrers, full, null, null);
    5454        Main.worker.submit(task);
    55         Main.worker.submit(new Runnable() {
    56             @Override
    57             public void run() {
     55        Main.worker.submit(() -> {
    5856                final List<PrimitiveId> downloaded = task.getDownloadedId();
    5957                if (downloaded != null) {
    60                     GuiHelper.runInEDT(new Runnable() {
    61                         @Override
    62                         public void run() {
    63                             Main.getLayerManager().getEditDataSet().setSelected(downloaded);
    64                         }
    65                     });
     58                    GuiHelper.runInEDT(() -> Main.getLayerManager().getEditDataSet().setSelected(downloaded));
    6659                }
    67             }
    6860        });
    6961    }

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

    r10428 r10601  
    293293
    294294    protected static void waitFuture(final Future<?> future, final PleaseWaitProgressMonitor monitor) {
    295         Main.worker.submit(
    296                 new Runnable() {
    297                     @Override
    298                     public void run() {
     295        Main.worker.submit(() -> {
    299296                        try {
    300297                            future.get();
    … …  
    304301                        }
    305302                        monitor.close();
    306                     }
    307                 }
    308         );
     303                    });
    309304    }
    310305

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

    r10181 r10601  
    1919import org.openstreetmap.josm.data.Bounds;
    2020import org.openstreetmap.josm.data.coor.LatLon;
    21 import org.openstreetmap.josm.gui.MapFrame;
    22 import org.openstreetmap.josm.gui.MapFrameListener;
    2321import org.openstreetmap.josm.gui.MapView;
    2422import org.openstreetmap.josm.gui.dialogs.LatLonDialog;
    … …  
    223221        super.installAdapters();
    224222        // make this action listen to mapframe change events
    225         Main.addMapFrameListener(new MapFrameListener() {
    226             @Override
    227             public void mapFrameInitialized(MapFrame oldFrame, MapFrame newFrame) {
    228                 updateEnabledState();
    229             }
    230         });
     223        Main.addMapFrameListener((o, n) -> updateEnabledState());
    231224    }
    232225}

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

    r10436 r10601  
    4141        if (targetLayer == null)
    4242            return;
    43         Main.worker.submit(new Runnable() {
    44             @Override
    45             public void run() {
     43        Main.worker.submit(() -> {
    4644                boolean layerMerged = false;
    4745                for (final Layer sourceLayer: sourceLayers) {
    … …  
    5452                        }
    5553                        targetLayer.mergeFrom(sourceLayer);
    56                         GuiHelper.runInEDTAndWait(new Runnable() {
    57                             @Override
    58                             public void run() {
    59                                 Main.getLayerManager().removeLayer(sourceLayer);
    60                             }
    61                         });
     54                        GuiHelper.runInEDTAndWait(() -> Main.getLayerManager().removeLayer(sourceLayer));
    6255                        layerMerged = true;
    6356                    }
    … …  
    6659                    Main.getLayerManager().setActiveLayer(targetLayer);
    6760                }
    68             }
    6961        });
    7062    }
    … …  
    10092    @Override
    10193    protected void updateEnabledState() {
    102         GuiHelper.runInEDT(new Runnable() {
    103             @Override
    104             public void run() {
     94        GuiHelper.runInEDT(() -> {
    10595                final Layer sourceLayer = getSourceLayer();
    10696                if (sourceLayer == null) {
    … …  
    110100                    setEnabled(!possibleMergeTargets.isEmpty());
    111101                }
    112             }
    113102        });
    114103    }

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

    r10407 r10601  
    1010import java.io.BufferedReader;
    1111import java.io.File;
    12 import java.io.FilenameFilter;
    1312import java.io.IOException;
    1413import java.nio.charset.StandardCharsets;
    … …  
    127126                } else if (file.getParentFile() != null) {
    128127                    // try to guess an extension using the specified fileFilter
    129                     final File[] matchingFiles = file.getParentFile().listFiles(new FilenameFilter() {
    130                         @Override
    131                         public boolean accept(File dir, String name) {
    132                             return name.startsWith(file.getName())
    133                                     && fileFilter != null && fileFilter.accept(new File(dir, name));
    134                         }
    135                     });
     128                    final File[] matchingFiles = file.getParentFile().listFiles((dir, name) ->
     129                            name.startsWith(file.getName()) && fileFilter != null && fileFilter.accept(new File(dir, name)));
    136130                    if (matchingFiles != null && matchingFiles.length == 1) {
    137131                        // use the unique match as filename
    … …  
    260254                    if (!chosenImporter.acceptFile(f)) {
    261255                        if (f.isDirectory()) {
    262                             SwingUtilities.invokeLater(new Runnable() {
    263                                 @Override
    264                                 public void run() {
    265                                     JOptionPane.showMessageDialog(Main.parent, tr(
    266                                             "<html>Cannot open directory ''{0}''.<br>Please select a file.</html>",
    267                                             f.getAbsolutePath()), tr("Open file"), JOptionPane.ERROR_MESSAGE);
    268                                 }
    269                             });
     256                            SwingUtilities.invokeLater(() -> JOptionPane.showMessageDialog(Main.parent, tr(
     257                                    "<html>Cannot open directory ''{0}''.<br>Please select a file.</html>",
     258                                    f.getAbsolutePath()), tr("Open file"), JOptionPane.ERROR_MESSAGE));
    270259                            // TODO when changing to Java 6: Don't cancel the task here but use different modality. (Currently 2 dialogs
    271260                            // would block each other.)

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

    r10413 r10601  
    1414import java.util.Collection;
    1515import java.util.Collections;
    16 import java.util.Comparator;
    1716import java.util.HashSet;
    1817import java.util.List;
    … …  
    245244                            ImageProvider.get("warning-small"), JLabel.LEFT), GBC.eol().fill(GBC.HORIZONTAL));
    246245
    247             Collections.sort(toPurgeAdditionally, new Comparator<OsmPrimitive>() {
    248                 @Override
    249                 public int compare(OsmPrimitive o1, OsmPrimitive o2) {
    250                     int type = o2.getType().compareTo(o1.getType());
    251                     if (type != 0)
    252                         return type;
    253                     return Long.compare(o1.getUniqueId(), o2.getUniqueId());
    254                 }
     246            Collections.sort(toPurgeAdditionally, (o1, o2) -> {
     247                int type = o2.getType().compareTo(o1.getType());
     248                if (type != 0)
     249                    return type;
     250                return Long.compare(o1.getUniqueId(), o2.getUniqueId());
    255251            });
    256252            JList<OsmPrimitive> list = new JList<>(toPurgeAdditionally.toArray(new OsmPrimitive[toPurgeAdditionally.size()]));

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

    r10432 r10601  
    112112        @Override
    113113        protected void finish() {
    114             SwingUtilities.invokeLater(new Runnable() {
    115                 @Override
    116                 public void run() {
    117                     if (canceled)
    118                         return;
    119                     addLayers();
    120                     runPostLoadTasks();
    121                 }
     114            SwingUtilities.invokeLater(() -> {
     115                if (canceled)
     116                    return;
     117                addLayers();
     118                runPostLoadTasks();
    122119            });
    123120        }

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

    r10599 r10601  
    2727import javax.swing.JPanel;
    2828import javax.swing.ListSelectionModel;
    29 import javax.swing.event.ListSelectionEvent;
    30 import javax.swing.event.ListSelectionListener;
    3129
    3230import org.openstreetmap.josm.Main;
    … …  
    252250        private void configureList() {
    253251            list.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
    254             list.addListSelectionListener(new ListSelectionListener() {
    255                 @Override
    256                 public void valueChanged(ListSelectionEvent e) {
    257                     final Way selected = list.getSelectedValue();
    258                     if (Main.isDisplayingMapView() && selected != null && selected.getNodesCount() > 1) {
    259                         final Collection<WaySegment> segments = new ArrayList<>(selected.getNodesCount() - 1);
    260                         final Iterator<Node> it = selected.getNodes().iterator();
    261                         Node previousNode = it.next();
    262                         while (it.hasNext()) {
    263                             final Node node = it.next();
    264                             segments.add(WaySegment.forNodePair(selectedWay, previousNode, node));
    265                             previousNode = node;
    266                         }
    267                         setHighlightedWaySegments(segments);
     252            list.addListSelectionListener(e -> {
     253                final Way selected = list.getSelectedValue();
     254                if (Main.isDisplayingMapView() && selected != null && selected.getNodesCount() > 1) {
     255                    final Collection<WaySegment> segments = new ArrayList<>(selected.getNodesCount() - 1);
     256                    final Iterator<Node> it = selected.getNodes().iterator();
     257                    Node previousNode = it.next();
     258                    while (it.hasNext()) {
     259                        final Node node = it.next();
     260                        segments.add(WaySegment.forNodePair(selectedWay, previousNode, node));
     261                        previousNode = node;
    268262                    }
     263                    setHighlightedWaySegments(segments);
    269264                }
    270265            });

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

    r10448 r10601  
    4343import org.openstreetmap.josm.tools.GBC;
    4444import org.openstreetmap.josm.tools.ImageProvider;
    45 import org.openstreetmap.josm.tools.Predicate;
    4645import org.openstreetmap.josm.tools.Shortcut;
    4746import org.openstreetmap.josm.tools.UserCancelException;
    … …  
    251250
    252251        private static boolean isTagged(final Iterable<Node> existingNodes) {
    253             return Utils.exists(existingNodes, new Predicate<Node>() {
    254                 @Override
    255                 public boolean evaluate(final Node selectedNode) {
    256                     return selectedNode.hasKeys();
    257                 }
    258             });
     252            return Utils.exists(existingNodes, selectedNode -> selectedNode.hasKeys());
    259253        }
    260254
    261255        private static boolean isUsedInRelations(final Iterable<Node> existingNodes) {
    262             return Utils.exists(existingNodes, new Predicate<Node>() {
    263                 @Override
    264                 public boolean evaluate(final Node selectedNode) {
    265                     return Utils.exists(selectedNode.getReferrers(), OsmPrimitive.relationPredicate);
    266                 }
    267             });
     256            return Utils.exists(existingNodes, selectedNode -> Utils.exists(selectedNode.getReferrers(), OsmPrimitive.relationPredicate));
    268257        }
    269258

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

    r10548 r10601  
    289289                return;
    290290            }
    291             Runnable r = new Runnable() {
    292                 @Override
    293                 public void run() {
    294                     processPostParentChecker(layer, toUpload);
    295                 }
    296             };
    297             SwingUtilities.invokeLater(r);
     291            SwingUtilities.invokeLater(() -> processPostParentChecker(layer, toUpload));
    298292        }
    299293

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

    r10446 r10601  
    142142            // update GUI on Swing EDT
    143143            //
    144             GuiHelper.runInEDT(new Runnable() {
    145                 @Override
    146                 public void run() {
    147                     Main.map.validatorDialog.tree.setErrors(errors);
    148                     Main.map.validatorDialog.unfurlDialog();
    149                     Main.getLayerManager().getEditDataSet().fireSelectionChanged();
    150                 }
     144            GuiHelper.runInEDT(() -> {
     145                Main.map.validatorDialog.tree.setErrors(errors);
     146                Main.map.validatorDialog.unfurlDialog();
     147                Main.getLayerManager().getEditDataSet().fireSelectionChanged();
    151148            });
    152149        }

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/AbstractChangesetDownloadTask.java

    r10250 r10601  
    6161            // Run on the EDT because UI updates are triggered.
    6262            //
    63             Runnable r = new Runnable() {
    64                 @Override public void run() {
    65                     ChangesetCache.getInstance().update(downloadedChangesets);
    66                 }
    67             };
     63            Runnable r = () -> ChangesetCache.getInstance().update(downloadedChangesets);
    6864            if (SwingUtilities.isEventDispatchThread()) {
    6965                r.run();

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadReferrersTask.java

    r10467 r10601  
    117117        DataSetMerger visitor = new DataSetMerger(targetLayer.data, parents);
    118118        visitor.merge();
    119         SwingUtilities.invokeLater(
    120                 new Runnable() {
    121                     @Override
    122                     public void run() {
    123                         targetLayer.onPostDownloadFromServer();
    124                     }
    125                 }
    126         );
     119        SwingUtilities.invokeLater(targetLayer::onPostDownloadFromServer);
    127120        if (visitor.getConflicts().isEmpty())
    128121            return;
    … …  
    131124                Main.parent,
    132125                trn("There was {0} conflict during import.",
    133                         "There were {0} conflicts during import.",
    134                         visitor.getConflicts().size(),
    135                         visitor.getConflicts().size()
     126                    "There were {0} conflicts during import.",
     127                    visitor.getConflicts().size(),
     128                    visitor.getConflicts().size()
    136129                ),
    137130                trn("Conflict during download", "Conflicts during download", visitor.getConflicts().size()),

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadTaskList.java

    r10436 r10601  
    3333import org.openstreetmap.josm.gui.layer.OsmDataLayer;
    3434import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    35 import org.openstreetmap.josm.gui.progress.ProgressMonitor.CancelListener;
    3635import org.openstreetmap.josm.gui.util.GuiHelper;
    3736import org.openstreetmap.josm.tools.ExceptionUtil;
    … …  
    8887            }
    8988        }
    90         progressMonitor.addCancelListener(new CancelListener() {
    91             @Override
    92             public void operationCanceled() {
    93                 for (DownloadTask dt : tasks) {
    94                     dt.cancel();
    95                 }
     89        progressMonitor.addCancelListener(() -> {
     90            for (DownloadTask dt : tasks) {
     91                dt.cancel();
    9692            }
    9793        });
    … …  
    151147            }
    152148        }
    153         EventQueue.invokeLater(new Runnable() {
    154             @Override public void run() {
    155                 UpdateSelectionAction.updatePrimitives(toSelect);
    156             }
    157         });
     149        EventQueue.invokeLater(() -> UpdateSelectionAction.updatePrimitives(toSelect));
    158150    }
    159151
    … …  
    270262                }
    271263
    272                 GuiHelper.runInEDT(new Runnable() {
    273                     @Override
    274                     public void run() {
    275                         if (items.size() == 1 && tr("No data found in this area.").equals(items.iterator().next())) {
    276                             new Notification(items.iterator().next()).setIcon(JOptionPane.WARNING_MESSAGE).show();
    277                         } else {
    278                             JOptionPane.showMessageDialog(Main.parent, "<html>"
    279                                     + tr("The following errors occurred during mass download: {0}",
    280                                             Utils.joinAsHtmlUnorderedList(items)) + "</html>",
    281                                     tr("Errors during download"), JOptionPane.ERROR_MESSAGE);
    282                         }
     264                GuiHelper.runInEDT(() -> {
     265                    if (items.size() == 1 && tr("No data found in this area.").equals(items.iterator().next())) {
     266                        new Notification(items.iterator().next()).setIcon(JOptionPane.WARNING_MESSAGE).show();
     267                    } else {
     268                        JOptionPane.showMessageDialog(Main.parent, "<html>"
     269                                + tr("The following errors occurred during mass download: {0}",
     270                                        Utils.joinAsHtmlUnorderedList(items)) + "</html>",
     271                                tr("Errors during download"), JOptionPane.ERROR_MESSAGE);
    283272                    }
    284273                });
    … …  
    312301                }
    313302                if (!myPrimitives.isEmpty()) {
    314                     GuiHelper.runInEDT(new Runnable() {
    315                         @Override public void run() {
    316                             handlePotentiallyDeletedPrimitives(myPrimitives);
    317                         }
    318                     });
     303                    GuiHelper.runInEDT(() -> handlePotentiallyDeletedPrimitives(myPrimitives));
    319304                }
    320305            }

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/PostDownloadHandler.java

    r10212 r10601  
    5858            final Object error = errors.iterator().next();
    5959            if (!GraphicsEnvironment.isHeadless()) {
    60                 SwingUtilities.invokeLater(new Runnable() {
    61                     @Override
    62                     public void run() {
    63                         if (error instanceof Exception) {
    64                             ExceptionDialogUtil.explainException((Exception) error);
    65                         } else if (tr("No data found in this area.").equals(error)) {
    66                             new Notification(error.toString()).setIcon(JOptionPane.WARNING_MESSAGE).show();
    67                         } else {
    68                             JOptionPane.showMessageDialog(
    69                                     Main.parent,
    70                                     error.toString(),
    71                                     tr("Error during download"),
    72                                     JOptionPane.ERROR_MESSAGE);
    73                         }
     60                SwingUtilities.invokeLater(() -> {
     61                    if (error instanceof Exception) {
     62                        ExceptionDialogUtil.explainException((Exception) error);
     63                    } else if (tr("No data found in this area.").equals(error)) {
     64                        new Notification(error.toString()).setIcon(JOptionPane.WARNING_MESSAGE).show();
     65                    } else {
     66                        JOptionPane.showMessageDialog(
     67                                Main.parent,
     68                                error.toString(),
     69                                tr("Error during download"),
     70                                JOptionPane.ERROR_MESSAGE);
    7471                    }
    7572                });
    … …  
    9188
    9289            if (!GraphicsEnvironment.isHeadless()) {
    93                 SwingUtilities.invokeLater(new Runnable() {
    94                     @Override
    95                     public void run() {
    96                         JOptionPane.showMessageDialog(
    97                                 Main.parent,
    98                                 "<html>"+Utils.joinAsHtmlUnorderedList(items)+"</html>",
    99                                 tr("Errors during download"),
    100                                 JOptionPane.ERROR_MESSAGE);
    101                     }
    102                 });
     90                SwingUtilities.invokeLater(() -> JOptionPane.showMessageDialog(
     91                        Main.parent,
     92                        "<html>"+Utils.joinAsHtmlUnorderedList(items)+"</html>",
     93                        tr("Errors during download"),
     94                        JOptionPane.ERROR_MESSAGE));
    10395            }
    10496            return;

  • trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java

    r10448 r10601  
    99import java.awt.Point;
    1010import java.awt.Rectangle;
    11 import java.awt.event.ActionEvent;
    12 import java.awt.event.ActionListener;
    1311import java.awt.event.KeyEvent;
    1412import java.awt.event.MouseEvent;
    … …  
    447445            useLastMoveCommandIfPossible();
    448446            // Schedule a timer to update status line "initialMoveDelay+1" ms in the future
    449             GuiHelper.scheduleTimer(initialMoveDelay+1, new ActionListener() {
    450                 @Override
    451                 public void actionPerformed(ActionEvent evt) {
    452                     updateStatusLine();
    453                 }
    454             }, false);
     447            GuiHelper.scheduleTimer(initialMoveDelay+1, evt -> updateStatusLine(), false);
    455448            break;
    456449        case SELECT:
    … …  
    616609                        // We need to do it like this as otherwise drawAction will see a double
    617610                        // click and switch back to SelectMode
    618                         Main.worker.execute(new Runnable() {
    619                             @Override
    620                             public void run() {
    621                                 Main.map.selectDrawTool(true);
    622                             }
    623                         });
     611                        Main.worker.execute(() -> Main.map.selectDrawTool(true));
    624612                        return;
    625613                    }

  • trunk/src/org/openstreetmap/josm/actions/relation/AddSelectionToRelations.java

    r10446 r10601  
    6363    @Override
    6464    public void selectionChanged(final Collection<? extends OsmPrimitive> newSelection) {
    65         GuiHelper.runInEDT(new Runnable() {
    66             @Override
    67             public void run() {
    68                 setEnabled(newSelection != null && !newSelection.isEmpty());
    69             }
    70         });
     65        GuiHelper.runInEDT(() -> setEnabled(newSelection != null && !newSelection.isEmpty()));
    7166    }
    7267}

  • trunk/src/org/openstreetmap/josm/actions/relation/DownloadMembersAction.java

    r10413 r10601  
    1010import org.openstreetmap.josm.Main;
    1111import org.openstreetmap.josm.data.osm.OsmPrimitive;
    12 import org.openstreetmap.josm.data.osm.Relation;
    1312import org.openstreetmap.josm.gui.dialogs.relation.DownloadRelationTask;
    1413import org.openstreetmap.josm.io.OnlineResource;
    1514import org.openstreetmap.josm.tools.ImageProvider;
    16 import org.openstreetmap.josm.tools.Predicate;
    1715import org.openstreetmap.josm.tools.Utils;
    1816
    … …  
    4240    public void setPrimitives(Collection<? extends OsmPrimitive> primitives) {
    4341        // selected non-new relations
    44         this.relations = Utils.filter(getRelations(primitives), new Predicate<Relation>() {
    45             @Override
    46             public boolean evaluate(Relation r) {
    47                 return !r.isNew();
    48             }
    49         });
     42        this.relations = Utils.filter(getRelations(primitives), r -> !r.isNew());
    5043        updateEnabledState();
    5144    }

  • trunk/src/org/openstreetmap/josm/actions/relation/DownloadSelectedIncompleteMembersAction.java

    r10413 r10601  
    1515import org.openstreetmap.josm.io.OnlineResource;
    1616import org.openstreetmap.josm.tools.ImageProvider;
    17 import org.openstreetmap.josm.tools.Predicate;
    1817import org.openstreetmap.josm.tools.Utils;
    1918
    … …  
    4342        Set<OsmPrimitive> ret = new HashSet<>();
    4443        for (Relation r : rels) {
    45             ret.addAll(Utils.filter(r.getIncompleteMembers(), new Predicate<OsmPrimitive>() {
    46                 @Override
    47                 public boolean evaluate(OsmPrimitive osm) {
    48                     return !osm.isNew();
    49                 }
    50             }));
     44            ret.addAll(Utils.filter(r.getIncompleteMembers(), osm -> !osm.isNew()));
    5145        }
    5246        return ret;
    … …  
    6559    public void setPrimitives(Collection<? extends OsmPrimitive> primitives) {
    6660        // selected relations with incomplete members
    67         this.relations = Utils.filter(getRelations(primitives), new Predicate<Relation>() {
    68             @Override
    69             public boolean evaluate(Relation r) {
    70                 return r.hasIncompleteMembers();
    71             }
    72         });
     61        this.relations = Utils.filter(getRelations(primitives), r -> r.hasIncompleteMembers());
    7362        this.incompleteMembers = buildSetOfIncompleteMembers(relations);
    7463        updateEnabledState();

  • trunk/src/org/openstreetmap/josm/actions/search/SearchAction.java

    r10600 r10601  
    651651        private static SearchTask newSearchTask(SearchSetting setting, final DataSet ds, SearchReceiver resultReceiver) {
    652652            final Collection<OsmPrimitive> selection = new HashSet<>(ds.getAllSelected());
    653             return new SearchTask(ds, setting, selection, new Predicate<OsmPrimitive>() {
    654                 @Override
    655                 public boolean evaluate(OsmPrimitive o) {
    656                     return ds.isSelected(o);
    657                 }
    658             }, resultReceiver);
     653            return new SearchTask(ds, setting, selection, ds::isSelected, resultReceiver);
    659654        }
    660655

  • trunk/src/org/openstreetmap/josm/actions/upload/FixDataHook.java

    r10600 r10601  
    4646        deprecated.add(new FixDataTag("highway", "stile", "barrier", "stile"));
    4747        // CHECKSTYLE.ON: SingleSpaceSeparator
    48         deprecated.add(new FixData() {
    49             @Override
    50             public boolean fixKeys(Map<String, String> keys, OsmPrimitive osm) {
    51                 if (osm instanceof Relation && "multipolygon".equals(keys.get("type")) && "administrative".equals(keys.get("boundary"))) {
    52                     keys.put("type", "boundary");
    53                     return true;
    54                 }
    55                 return false;
    56             }
     48        deprecated.add((keys, osm) -> {
     49            if (osm instanceof Relation && "multipolygon".equals(keys.get("type")) && "administrative".equals(keys.get("boundary"))) {
     50                keys.put("type", "boundary");
     51                return true;
     52            }
     53            return false;
    5754        });
    5855    }
Note: See TracChangeset for help on using the changeset viewer.