Changeset 13849 in josm for trunk/src/org/openstreetmap/josm


Ignore:
Timestamp:
2018-05-26T19:36:19+02:00 (15 months ago)
Author:
Don-vip
Message:

SonarQube - fix minor code issues

Location:
trunk/src/org/openstreetmap/josm
Files:
1 added
33 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/ChangesetContentDownloadTask.java

    r12620 r13849  
    7878                setCanceled(true);
    7979                Logging.trace(e);
    80                 return;
    8180            } catch (OsmTransferException e) {
    82                 if (isCanceled())
    83                     return;
    84                 rememberLastException(e);
     81                if (!isCanceled()) {
     82                    rememberLastException(e);
     83                }
    8584            }
    8685        }
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/PostDownloadHandler.java

    r12620 r13849  
    120120                        JOptionPane.ERROR_MESSAGE));
    121121            }
    122             return;
    123122        }
    124123    }
  • trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java

    r13444 r13849  
    275275        }
    276276
    277         mv.setNewCursor(getCursor(c), this);
     277        mv.setNewCursor(getCursor(c.orElse(null)), this);
    278278
    279279        // return early if there can't be any highlights
     
    296296     * features. The only exception is the "move" cursor when actually dragging
    297297     * primitives.
    298      * @param nearbyStuff  primitives near the cursor
     298     * @param nearbyStuff primitives near the cursor
    299299     * @return the cursor that should be displayed
    300300     */
    301     private Cursor getCursor(Optional<OsmPrimitive> nearbyStuff) {
     301    private Cursor getCursor(OsmPrimitive nearbyStuff) {
    302302        String c = "rect";
    303303        switch(mode) {
     
    307307                break;
    308308            }
    309             final OsmPrimitive osm = nearbyStuff.orElse(null);
     309            final OsmPrimitive osm = nearbyStuff;
    310310
    311311            if (dragInProgress()) {
     
    516516                needsRepaint = true;
    517517            }
    518             mv.setNewCursor(getCursor(Optional.ofNullable(p)), this);
     518            mv.setNewCursor(getCursor(p), this);
    519519            // also update the stored mouse event, so we can display the correct cursor
    520520            // when dragging a node onto another one and then press CTRL to merge
  • trunk/src/org/openstreetmap/josm/data/Preferences.java

    r13838 r13849  
    406406                continue;
    407407            Utils.instanceOfAndCast(e.getValue(), ListSetting.class)
    408                     .map(d -> d.getValue())
     408                    .map(ListSetting::getValue)
    409409                    .map(lst -> ColorInfo.fromPref(lst, false))
    410410                    .ifPresent(info -> all.put(e.getKey(), info));
     
    414414                continue;
    415415            Utils.instanceOfAndCast(e.getValue(), ListSetting.class)
    416                     .map(d -> d.getValue())
     416                    .map(ListSetting::getValue)
    417417                    .map(lst -> ColorInfo.fromPref(lst, true))
    418418                    .ifPresent(infoDef -> {
  • trunk/src/org/openstreetmap/josm/data/projection/ProjectionCLI.java

    r13207 r13849  
    3333    public static final ProjectionCLI INSTANCE = new ProjectionCLI();
    3434
    35     private boolean argInverse = false;         // NOPMD
    36     private boolean argSwitchInput = false;     // NOPMD
    37     private boolean argSwitchOutput = false;    // NOPMD
     35    private boolean argInverse;
     36    private boolean argSwitchInput;
     37    private boolean argSwitchOutput;
    3838
    3939    @Override
     
    7777            if (arg.isEmpty()) throw new IllegalArgumentException("non-empty argument expected");
    7878            if (arg.startsWith("+")) {
    79                 if (arg.equals("+to")) {
     79                if ("+to".equals(arg)) {
    8080                    toTokenSeen = true;
    8181                } else {
     
    142142        }
    143143
    144         if (files.isEmpty() || files.get(0).equals("-")) {
     144        if (files.isEmpty() || "-".equals(files.get(0))) {
    145145            processInput(fromProj, toProj, new BufferedReader(new InputStreamReader(System.in, Charset.defaultCharset())));
    146146        } else {
     
    174174    }
    175175
    176     private CustomProjection createProjection(String params) throws ProjectionConfigurationException {
     176    private static CustomProjection createProjection(String params) throws ProjectionConfigurationException {
    177177        CustomProjection proj = new CustomProjection();
    178178        proj.update(params);
  • trunk/src/org/openstreetmap/josm/data/validation/tests/WronglyOrderedWays.java

    r11129 r13849  
    3737
    3838        String natural = w.get("natural");
    39         if (natural == null)
     39        if (natural == null) {
    4040            return;
    41         else if ("coastline".equals(natural) && Geometry.isClockwise(w)) {
     41        } else if ("coastline".equals(natural) && Geometry.isClockwise(w)) {
    4242            reportError(w, tr("Reversed coastline: land not on left side"), WRONGLY_ORDERED_COAST);
    4343        } else if ("land".equals(natural) && Geometry.isClockwise(w)) {
    4444            reportError(w, tr("Reversed land: land not on left side"), WRONGLY_ORDERED_LAND);
    45         } else
    46             return;
    47 
     45        }
    4846    }
    4947
  • trunk/src/org/openstreetmap/josm/gui/ExceptionDialogUtil.java

    r13761 r13849  
    194194        if (ex != null) {
    195195            // Users should be able to submit a bug report for an invocation target exception
    196             //
    197196            BugReportExceptionHandler.handleException(ex);
    198             return;
    199197        }
    200198    }
  • trunk/src/org/openstreetmap/josm/gui/autofilter/AutoFilterRule.java

    r12414 r13849  
    55import java.util.Objects;
    66import java.util.function.Function;
     7import java.util.function.UnaryOperator;
    78
    89/**
     
    1920    private final int minZoomLevel;
    2021
    21     private Function<String, String> valueFormatter = s -> s;
     22    private UnaryOperator<String> valueFormatter = s -> s;
    2223
    2324    private Comparator<String> valueComparator = Comparator.comparingInt(s -> Integer.parseInt(valueFormatter.apply(s)));
     
    6364     * @throws NullPointerException if {@code valueFormatter} is null
    6465     */
    65     public AutoFilterRule setValueFormatter(Function<String, String> valueFormatter) {
     66    public AutoFilterRule setValueFormatter(UnaryOperator<String> valueFormatter) {
    6667        this.valueFormatter = Objects.requireNonNull(valueFormatter);
    6768        return this;
  • trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java

    r13821 r13849  
    604604                AbstractProperty<Color> prop = layer.getColorProperty();
    605605                Color c = prop == null ? null : prop.get();
    606                 if (c == null || !model.getLayers().stream()
     606                if (c == null || model.getLayers().stream()
    607607                        .map(Layer::getColorProperty)
    608608                        .filter(Objects::nonNull)
    609609                        .map(AbstractProperty::get)
    610                         .anyMatch(oc -> oc != null && !oc.equals(c))) {
     610                        .noneMatch(oc -> oc != null && !oc.equals(c))) {
    611611                    /* not more than one color, don't use coloring */
    612612                    label.setForeground(UIManager.getColor(isSelected ? "Table.selectionForeground" : "Table.foreground"));
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/DateValidator.java

    r12620 r13849  
    7878            String msg = "<html>The current value isn't a valid date.<br>" + getStandardTooltipText()+ "</html>";
    7979            feedbackInvalid(msg);
    80             return;
    8180        } else {
    8281            String msg = "<html>" + getStandardTooltipText() + "</html>";
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/TimeValidator.java

    r12620 r13849  
    8080            String msg = "<html>The current value isn't a valid time.<br>" + getStandardTooltipText() + "</html>";
    8181            feedbackInvalid(msg);
    82             return;
    8382        } else {
    8483            String msg = "<html>" + getStandardTooltipText() + "</html>";
  • trunk/src/org/openstreetmap/josm/gui/dialogs/layer/LayerVisibilityAction.java

    r12419 r13849  
    300300            Collection<? extends Layer> usedLayers = filterLayers(layers);
    301301            setVisible(!usedLayers.isEmpty());
    302             if (!usedLayers.stream().anyMatch(Layer::isVisible)) {
     302            if (usedLayers.stream().noneMatch(Layer::isVisible)) {
    303303                slider.setEnabled(false);
    304304            } else {
  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/AbstractCopyAction.java

    r13521 r13849  
    77import java.util.Set;
    88import java.util.TreeSet;
    9 import java.util.function.Function;
     9import java.util.function.IntFunction;
    1010import java.util.function.Supplier;
    1111
     
    2424
    2525    private final JTable tagTable;
    26     private final Function<Integer, String> keySupplier;
     26    private final IntFunction<String> keySupplier;
    2727    private final Supplier<Collection<? extends Tagged>> objectSupplier;
    2828
     
    3333     * @param objectSupplier a supplier which returns the selected tagged object(s)
    3434     */
    35     public AbstractCopyAction(JTable tagTable, Function<Integer, String> keySupplier, Supplier<Collection<? extends Tagged>> objectSupplier) {
     35    public AbstractCopyAction(JTable tagTable, IntFunction<String> keySupplier, Supplier<Collection<? extends Tagged>> objectSupplier) {
    3636        this.tagTable = Objects.requireNonNull(tagTable);
    3737        this.keySupplier = Objects.requireNonNull(keySupplier);
  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/CopyAllKeyValueAction.java

    r13521 r13849  
    99import java.util.List;
    1010import java.util.Map.Entry;
    11 import java.util.function.Function;
     11import java.util.function.IntFunction;
    1212import java.util.function.Supplier;
    1313
     
    3131     * @param objectSp a supplier which returns the selected tagged object(s)
    3232     */
    33     public CopyAllKeyValueAction(JTable tagTable, Function<Integer, String> keyFn, Supplier<Collection<? extends Tagged>> objectSp) {
     33    public CopyAllKeyValueAction(JTable tagTable, IntFunction<String> keyFn, Supplier<Collection<? extends Tagged>> objectSp) {
    3434        super(tagTable, keyFn, objectSp);
    3535        putValue(NAME, tr("Copy all Keys/Values"));
  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/CopyKeyValueAction.java

    r13521 r13849  
    66import java.util.Collection;
    77import java.util.Collections;
    8 import java.util.function.Function;
     8import java.util.function.IntFunction;
    99import java.util.function.Supplier;
    1010
     
    2626     * @param objectSp a supplier which returns the selected tagged object(s)
    2727     */
    28     public CopyKeyValueAction(JTable tagTable, Function<Integer, String> keyFn, Supplier<Collection<? extends Tagged>> objectSp) {
     28    public CopyKeyValueAction(JTable tagTable, IntFunction<String> keyFn, Supplier<Collection<? extends Tagged>> objectSp) {
    2929        super(tagTable, keyFn, objectSp);
    3030        putValue(NAME, tr("Copy selected Key(s)/Value(s)"));
  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/CopyValueAction.java

    r13521 r13849  
    66import java.util.Collection;
    77import java.util.Collections;
    8 import java.util.function.Function;
     8import java.util.function.IntFunction;
    99import java.util.function.Supplier;
    1010
     
    2525     * @param objectSp a supplier which returns the selected tagged object(s)
    2626     */
    27     public CopyValueAction(JTable tagTable, Function<Integer, String> keyFn, Supplier<Collection<? extends Tagged>> objectSp) {
     27    public CopyValueAction(JTable tagTable, IntFunction<String> keyFn, Supplier<Collection<? extends Tagged>> objectSp) {
    2828        super(tagTable, keyFn, objectSp);
    2929        putValue(NAME, tr("Copy Value"));
  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/HelpAction.java

    r13522 r13849  
    1414import java.util.Map;
    1515import java.util.Objects;
    16 import java.util.function.Function;
     16import java.util.function.IntFunction;
    1717
    1818import javax.swing.AbstractAction;
     
    3636public class HelpAction extends AbstractAction {
    3737    private final JTable tagTable;
    38     private final Function<Integer, String> tagKeySupplier;
    39     private final Function<Integer, Map<String, Integer>> tagValuesSupplier;
     38    private final IntFunction<String> tagKeySupplier;
     39    private final IntFunction<Map<String, Integer>> tagValuesSupplier;
    4040
    4141    private final JTable membershipTable;
    42     private final Function<Integer, Relation> memberValueSupplier;
     42    private final IntFunction<Relation> memberValueSupplier;
    4343
    4444    /**
     
    5050     * @param memberValueSupplier Finds the parent relation from given row of membership table. Can be null
    5151     */
    52     public HelpAction(JTable tagTable, Function<Integer, String> tagKeySupplier, Function<Integer, Map<String, Integer>> tagValuesSupplier,
    53             JTable membershipTable, Function<Integer, Relation> memberValueSupplier) {
     52    public HelpAction(JTable tagTable, IntFunction<String> tagKeySupplier, IntFunction<Map<String, Integer>> tagValuesSupplier,
     53            JTable membershipTable, IntFunction<Relation> memberValueSupplier) {
    5454        this.tagTable = Objects.requireNonNull(tagTable);
    5555        this.tagKeySupplier = Objects.requireNonNull(tagKeySupplier);
  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/TaginfoAction.java

    r13521 r13849  
    77import java.util.Map;
    88import java.util.Objects;
    9 import java.util.function.Function;
     9import java.util.function.IntFunction;
    1010
    1111import javax.swing.JTable;
     
    2626
    2727    private final JTable tagTable;
    28     private final Function<Integer, String> tagKeySupplier;
    29     private final Function<Integer, Map<String, Integer>> tagValuesSupplier;
     28    private final IntFunction<String> tagKeySupplier;
     29    private final IntFunction<Map<String, Integer>> tagValuesSupplier;
    3030
    3131    private final JTable membershipTable;
    32     private final Function<Integer, Relation> memberValueSupplier;
     32    private final IntFunction<Relation> memberValueSupplier;
    3333
    3434    /**
     
    4040     * @param memberValueSupplier Finds the parent relation from given row of membership table. Can be null
    4141     */
    42     public TaginfoAction(JTable tagTable, Function<Integer, String> tagKeySupplier, Function<Integer, Map<String, Integer>> tagValuesSupplier,
    43             JTable membershipTable, Function<Integer, Relation> memberValueSupplier) {
     42    public TaginfoAction(JTable tagTable, IntFunction<String> tagKeySupplier, IntFunction<Map<String, Integer>> tagValuesSupplier,
     43            JTable membershipTable, IntFunction<Relation> memberValueSupplier) {
    4444        super(tr("Go to Taginfo"), "dialogs/taginfo", tr("Launch browser with Taginfo statistics for selected object"), null, false);
    4545        this.tagTable = Objects.requireNonNull(tagTable);
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/actions/AddFromSelectionAction.java

    r12846 r13849  
    5252                    ret.add(primitive);
    5353                }
    54                 continue;
    5554            } else {
    5655                ret.add(primitive);
  • trunk/src/org/openstreetmap/josm/gui/history/HistoryLoadTask.java

    r12713 r13849  
    174174        } catch (OsmTransferException e) {
    175175            lastException = e;
    176             return;
    177176        }
    178177    }
  • trunk/src/org/openstreetmap/josm/gui/history/TagInfoViewer.java

    r13521 r13849  
    88import java.util.HashMap;
    99import java.util.Map;
    10 import java.util.function.Function;
     10import java.util.function.IntFunction;
    1111import java.util.function.Supplier;
    1212
     
    8080        JPopupMenu tagMenu = new JPopupMenu();
    8181
    82         Function<Integer, String> tagKeyFn = x -> (String) table.getValueAt(x, 0);
    83         Function<Integer, Map<String, Integer>> tagValuesFn = x -> {
     82        IntFunction<String> tagKeyFn = x -> (String) table.getValueAt(x, 0);
     83        IntFunction<Map<String, Integer>> tagValuesFn = x -> {
    8484            Map<String, Integer> map = new HashMap<>();
    8585            String key = tagTableModel.getValue((String) table.getValueAt(x, 0));
  • trunk/src/org/openstreetmap/josm/gui/io/CredentialDialog.java

    r12992 r13849  
    402402                if (currentTF.getText().trim().isEmpty()) {
    403403                    currentTF.selectAll();
    404                     return;
    405404                } else if (nextTF.getText().trim().isEmpty()) {
    406405                    nextTF.requestFocusInWindow();
    407406                    nextTF.selectAll();
    408                     return;
    409407                } else {
    410408                    owner.new OKAction().actionPerformed(null);
  • trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java

    r13281 r13849  
    148148    public Object getInfoComponent() {
    149149        StringBuilder info = new StringBuilder(128)
    150                 .append("<html><head><style>")
    151                 .append("td { padding: 4px 16px; }")
    152                 .append("</style></head><body>");
     150                .append("<html><head><style>td { padding: 4px 16px; }</style></head><body>");
    153151
    154152        if (data.attr.containsKey("name")) {
  • trunk/src/org/openstreetmap/josm/gui/mappaint/RenderingCLI.java

    r13399 r13849  
    1515import java.util.Locale;
    1616import java.util.Optional;
    17 import java.util.function.Supplier;
     17import java.util.function.DoubleSupplier;
    1818import java.util.logging.Level;
    1919
     
    238238                break;
    239239            case 'b':
    240                 if (!getopt.getOptarg().equals("auto")) {
     240                if (!"auto".equals(getopt.getOptarg())) {
    241241                    try {
    242242                        argBounds = new Bounds(getopt.getOptarg(), ",", Bounds.ParseMethod.LEFT_BOTTOM_RIGHT_TOP, false);
     
    459459                EastNorth projAnchor = proj.latlon2eastNorth(argAnchor);
    460460
    461                 Double enPerMeter = null;
    462                 Supplier<Double> getEnPerMeter = () -> {
     461                double enPerMeter = Double.NaN;
     462                DoubleSupplier getEnPerMeter = () -> {
    463463                    double shiftMeter = 10;
    464464                    EastNorth projAnchorShifted = projAnchor.add(
     
    470470                if (scale == null) {
    471471                    if (argScale != null) {
    472                         enPerMeter = getEnPerMeter.get();
     472                        enPerMeter = getEnPerMeter.getAsDouble();
    473473                        scale = argScale * enPerMeter / PIXEL_PER_METER;
    474474                    } else if (argWidthM != null && argWidthPx != null) {
    475                         enPerMeter = getEnPerMeter.get();
     475                        enPerMeter = getEnPerMeter.getAsDouble();
    476476                        scale = argWidthM / argWidthPx * enPerMeter;
    477477                    } else if (argHeightM != null && argHeightPx != null) {
    478                         enPerMeter = getEnPerMeter.get();
     478                        enPerMeter = getEnPerMeter.getAsDouble();
    479479                        scale = argHeightM / argHeightPx * enPerMeter;
    480480                    } else {
     
    486486                double widthEn;
    487487                if (argWidthM != null) {
    488                     enPerMeter = Optional.ofNullable(enPerMeter).orElseGet(getEnPerMeter);
     488                    if (enPerMeter == Double.NaN) {
     489                        enPerMeter = getEnPerMeter.getAsDouble();
     490                    }
    489491                    widthEn = argWidthM * enPerMeter;
    490492                } else if (argWidthPx != null) {
     
    497499                double heightEn;
    498500                if (argHeightM != null) {
    499                     enPerMeter = Optional.ofNullable(enPerMeter).orElseGet(getEnPerMeter);
     501                    if (enPerMeter == Double.NaN) {
     502                        enPerMeter = getEnPerMeter.getAsDouble();
     503                    }
    500504                    heightEn = argHeightM * enPerMeter;
    501505                } else if (argHeightPx != null) {
  • trunk/src/org/openstreetmap/josm/gui/oauth/RetrieveAccessTokenTask.java

    r12620 r13849  
    8888        } catch (OsmTransferCanceledException e) {
    8989            Logging.trace(e);
    90             return;
    9190        } catch (final OsmOAuthAuthorizationException e) {
    9291            Logging.error(e);
  • trunk/src/org/openstreetmap/josm/gui/oauth/RetrieveRequestTokenTask.java

    r12620 r13849  
    8282        } catch (OsmTransferCanceledException e) {
    8383            Logging.trace(e);
    84             return;
    8584        } catch (final OsmOAuthAuthorizationException e) {
    8685            Logging.error(e);
  • trunk/src/org/openstreetmap/josm/gui/preferences/display/ColorPreference.java

    r13761 r13849  
    211211
    212212        @Override
    213         public boolean isCellEditable(int rowIndex, int columnIndex) {
    214             return false;
    215         }
    216 
    217         @Override
    218213        public void setValueAt(Object aValue, int rowIndex, int columnIndex) {
    219214            if (columnIndex == 1 && aValue instanceof Color) {
     
    386381    }
    387382
    388     private boolean isRemoveColor(ColorEntry ce) {
    389         return ce.info.getCategory().equals(NamedColorProperty.COLOR_CATEGORY_LAYER);
     383    private static boolean isRemoveColor(ColorEntry ce) {
     384        return NamedColorProperty.COLOR_CATEGORY_LAYER.equals(ce.info.getCategory());
    390385    }
    391386
  • trunk/src/org/openstreetmap/josm/gui/preferences/server/ApiUrlTestTask.java

    r12620 r13849  
    202202            Logging.error(e);
    203203            alertConnectionFailed();
    204             return;
    205204        }
    206205    }
  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetNameTemplateList.java

    r13564 r13849  
    6060                        if (t.nameTemplateFilter.match(primitive))
    6161                            return t;
    62                         else {
    63                             continue;
    64                         }
    6562                    } else if (t.matches(type, primitive.getKeys(), false)) {
    6663                        return t;
  • trunk/src/org/openstreetmap/josm/gui/widgets/OsmIdTextField.java

    r12620 r13849  
    104104                // convert tokens to int skipping v-words (version v2 etc)
    105105                c = s.charAt(0);
    106                 if (c == 'v') {
    107                     continue;
    108                 } else {
     106                if (c != 'v') {
    109107                    try {
    110108                        ids.addAll(SimplePrimitiveId.multipleFromString(s));
  • trunk/src/org/openstreetmap/josm/io/OsmConnection.java

    r12992 r13849  
    120120            if (response.isCanceled()) {
    121121                cancel = true;
    122                 return;
    123122            } else {
    124123                String username = response.getUsername() == null ? "" : response.getUsername();
  • trunk/src/org/openstreetmap/josm/plugins/PluginClassLoader.java

    r13581 r13849  
    4747        // Add dependency only if not already present (directly or transitively through another one)
    4848        boolean result = !dependencies.contains(Objects.requireNonNull(dependency, "dependency"))
    49                 && !dependencies.stream().anyMatch(pcl -> pcl.dependencies.contains(dependency))
     49                && dependencies.stream().noneMatch(pcl -> pcl.dependencies.contains(dependency))
    5050                && dependencies.add(dependency);
    5151        if (result) {
  • trunk/src/org/openstreetmap/josm/tools/HiDPISupport.java

    r13202 r13849  
    192192        }
    193193        List<List<Image>> allVars = imgs.stream().map(HiDPISupport::getResolutionVariants).collect(Collectors.toList());
    194         int maxVariants = allVars.stream().mapToInt(lst -> lst.size()).max().getAsInt();
     194        int maxVariants = allVars.stream().mapToInt(List<Image>::size).max().getAsInt();
    195195        if (maxVariants == 1)
    196196            return processor.apply(imgs);
Note: See TracChangeset for help on using the changeset viewer.