Changeset 14435 in josm for trunk/src


Ignore:
Timestamp:
2018-11-20T01:51:27+01:00 (12 months ago)
Author:
Don-vip
Message:

fix SonarQube issues

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

Legend:

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

    r14415 r14435  
    2929public class ProjectionCLI implements CLIModule {
    3030
     31    /** The unique instance **/
    3132    public static final ProjectionCLI INSTANCE = new ProjectionCLI();
    3233
     
    4344    public void processArguments(String[] argArray) {
    4445        List<String> positionalArguments = new OptionParser("JOSM projection")
    45             .addFlagParameter("help", this::showHelp)
     46            .addFlagParameter("help", ProjectionCLI::showHelp)
    4647            .addShortAlias("help", "h")
    4748            .addFlagParameter("inverse", () -> argInverse = true)
     
    8485     * Displays help on the console
    8586     */
    86     private void showHelp() {
     87    private static void showHelp() {
    8788        System.out.println(getHelp());
    8889        System.exit(0);
  • trunk/src/org/openstreetmap/josm/gui/download/DownloadDialog.java

    r14418 r14435  
    676676            });
    677677
    678             addPropertyChangeListener(DIVIDER_LOCATION_PROPERTY, e -> {
    679                 heightAdjustedExplicitly = true;
    680             });
     678            addPropertyChangeListener(DIVIDER_LOCATION_PROPERTY, e -> heightAdjustedExplicitly = true);
    681679        }
    682680
  • trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java

    r14428 r14435  
    17761776
    17771777        /**
     1778         * Constructs a new {@code PrecacheTask}.
    17781779         * @param progressMonitor that will be notified about progess of the task
    17791780         * @param bufferY buffer Y in degrees around which to download tiles
  • trunk/src/org/openstreetmap/josm/tools/OptionParser.java

    r14419 r14435  
    168168                        parameter = split[1];
    169169                    } else {
    170                         if (toHandle.isEmpty() || toHandle.getFirst().equals("--")) {
     170                        if (toHandle.isEmpty() || "--".equals(toHandle.getFirst())) {
    171171                            throw new OptionParseException(tr("{0}: option ''{1}'' requires an argument", program));
    172172                        }
     
    198198                option.option.runFor(option.parameter);
    199199            } catch (OptionParseException e) {
    200                 String message;
     200                StringBuilder message = new StringBuilder();
    201201                // Just add a nicer error message
    202202                if (option.parameter == null) {
    203                     message = tr("{0}: Error while handling option ''{1}''", program, option.optionName);
     203                    message.append(tr("{0}: Error while handling option ''{1}''", program, option.optionName));
    204204                } else {
    205                     message = tr("{0}: Invalid value {2} for option ''{1}''", program, option.optionName,
    206                             option.parameter);
     205                    message.append(tr("{0}: Invalid value {2} for option ''{1}''", program, option.optionName,
     206                            option.parameter));
    207207                }
    208208                if (!e.getLocalizedMessage().isEmpty()) {
    209                     message += ": " + e.getLocalizedMessage().isEmpty();
     209                    message.append(": ").append(e.getLocalizedMessage().isEmpty());
    210210                }
    211                 throw new OptionParseException(message, e);
     211                throw new OptionParseException(message.toString(), e);
    212212            }
    213213        }
     
    261261    }
    262262
    263     protected abstract static class AvailableOption {
    264 
    265         public boolean requiresParameter() {
     263    protected interface AvailableOption {
     264
     265        /**
     266         * Determines if this option requires a parameter.
     267         * @return {@code true} if this option requires a parameter ({@code false} by default)
     268         */
     269        default boolean requiresParameter() {
    266270            return false;
    267271        }
    268272
    269         public OptionCount getRequiredCount() {
     273        /**
     274         * Determines how often this option may / must be specified on the command line.
     275         * @return how often this option may / must be specified on the command line
     276         */
     277        default OptionCount getRequiredCount() {
    270278            return OptionCount.OPTIONAL;
    271279        }
     
    275283         * @param parameter The parameter if {@link #requiresParameter()} is true, <code>null</code> otherwise.
    276284         */
    277         public abstract void runFor(String parameter);
    278 
     285        void runFor(String parameter);
    279286    }
    280287
     
    292299
    293300    /**
     301     * Exception thrown when an option cannot be parsed.
    294302     * @author Michael Zangl
    295303     */
     
    307315
    308316        /**
     317         * Create an error with a localized description
    309318         * @param localizedMessage The message to display to the user.
    310319         */
     
    315324
    316325        /**
     326         * Create an error with a localized description and a root cause
    317327         * @param localizedMessage The message to display to the user.
    318328         * @param t The error that caused this message to be displayed.
Note: See TracChangeset for help on using the changeset viewer.