Changeset 10936 in josm for trunk


Ignore:
Timestamp:
2016-09-03T03:27:18+02:00 (3 years ago)
Author:
Don-vip
Message:

sonar

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

Legend:

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

    r10899 r10936  
    593593
    594594        private final String name;
    595         private Runnable task;
     595        private final Runnable task;
    596596
    597597        protected InitializationTask(String name, Runnable task) {
  • trunk/src/org/openstreetmap/josm/actions/ImageryAdjustAction.java

    r10932 r10936  
    304304        }
    305305
    306         private void restoreMapModeState() {
    307             if (Main.map == null)
    308                 return;
    309             if (oldMapMode != null) {
    310                 Main.map.selectMapMode(oldMapMode);
    311                 oldMapMode = null;
    312             } else {
    313                 Main.map.selectSelectTool(false);
    314             }
    315         }
    316 
    317306        class WindowEventHandler extends WindowAdapter {
    318307            @Override
     
    321310                restoreMapModeState();
    322311            }
     312
     313            private void restoreMapModeState() {
     314                if (Main.map == null)
     315                    return;
     316                if (oldMapMode != null) {
     317                    Main.map.selectMapMode(oldMapMode);
     318                    oldMapMode = null;
     319                } else {
     320                    Main.map.selectSelectTool(false);
     321                }
     322            }
    323323        }
    324324    }
  • trunk/src/org/openstreetmap/josm/actions/ShowStatusReportAction.java

    r10933 r10936  
    9393            .append(runtimeVersion != null ? runtimeVersion : System.getProperty("java.version")).append(", ")
    9494            .append(System.getProperty("java.vendor")).append(", ")
    95             .append(System.getProperty("java.vm.name")).append('\n');
    96 
    97         text.append("Screen: ");
     95            .append(System.getProperty("java.vm.name"))
     96            .append("\nScreen: ");
    9897        if (!GraphicsEnvironment.isHeadless()) {
    9998            text.append(Arrays.stream(GraphicsEnvironment.getLocalGraphicsEnvironment().getScreenDevices()).map(gd -> {
  • trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

    r10875 r10936  
    640640    }
    641641
    642     private String getTitle(boolean newNode, Node n, Collection<OsmPrimitive> newSelection, List<Way> reuseWays,
     642    private static String getTitle(boolean newNode, Node n, Collection<OsmPrimitive> newSelection, List<Way> reuseWays,
    643643            boolean extendedWay) {
    644644        String title;
  • trunk/src/org/openstreetmap/josm/gui/ProgramArguments.java

    r10899 r10936  
    2929     * JOSM command line options.
    3030     * @see <a href="https://josm.openstreetmap.de/wiki/Help/CommandLineOptions">Help/CommandLineOptions</a>
    31      * @since xxx
    3231     */
    3332    public enum Option {
     
    102101     * Construct the program arguments object
    103102     * @param args The args passed to main.
    104      */
    105     public ProgramArguments(String[] args) {
     103     * @since 10936
     104     */
     105    public ProgramArguments(String ... args) {
    106106        Stream.of(Option.values()).forEach(o -> argMap.put(o, new ArrayList<>()));
    107 
    108107        buildCommandLineArgumentMap(args);
    109108    }
     
    216215    }
    217216
    218     private static String getValue(String[] kv) {
     217    private static String getValue(String ... kv) {
    219218        if (kv.length < 2) {
    220219            return "";
  • trunk/src/org/openstreetmap/josm/gui/datatransfer/AbstractStackTransferHandler.java

    r10881 r10936  
    44import java.awt.datatransfer.UnsupportedFlavorException;
    55import java.io.IOException;
     6import java.util.Arrays;
    67import java.util.Collection;
    78
     
    4950            if (df.supports(support)) {
    5051                try {
     52                    if (Main.isDebugEnabled()) {
     53                        Main.debug("{0} pasting {1} at {2}", df.getClass().getSimpleName(), Arrays.toString(support.getDataFlavors()), center);
     54                    }
    5155                    if (df.importData(support, layer, center)) {
    5256                        return true;
     
    5559                    Main.warn(e);
    5660                } catch (RuntimeException e) {
    57                     BugReport.intercept(e).put("paster", df).put("flavors", () -> support.getDataFlavors()).warn();
     61                    BugReport.intercept(e).put("paster", df).put("flavors", support::getDataFlavors).warn();
    5862                }
    5963            }
  • trunk/src/org/openstreetmap/josm/gui/datatransfer/OpenTransferHandler.java

    r10881 r10936  
    44import java.util.Arrays;
    55import java.util.Collection;
     6import java.util.Collections;
    67
    78import org.openstreetmap.josm.gui.datatransfer.importers.AbstractOsmDataPaster;
     
    2223    @Override
    2324    protected Collection<AbstractOsmDataPaster> getSupportedPasters() {
    24         return SUPPORTED;
     25        return Collections.unmodifiableCollection(SUPPORTED);
    2526    }
    2627}
  • trunk/src/org/openstreetmap/josm/gui/datatransfer/OsmTransferHandler.java

    r10890 r10936  
    88import java.util.Arrays;
    99import java.util.Collection;
     10import java.util.Collections;
    1011
    1112import org.openstreetmap.josm.Main;
     
    3536    @Override
    3637    protected Collection<AbstractOsmDataPaster> getSupportedPasters() {
    37         return SUPPORTED;
     38        return Collections.unmodifiableCollection(SUPPORTED);
    3839    }
    3940
  • trunk/src/org/openstreetmap/josm/io/protocols/data/Handler.java

    r10931 r10936  
    3232        String protocolHandlers = System.getProperty("java.protocol.handler.pkgs", "");
    3333        if (!protocolHandlers.contains(pkg)) {
    34             if (!protocolHandlers.isEmpty()) {
    35                 protocolHandlers += "|";
     34            StringBuilder sb = new StringBuilder(protocolHandlers);
     35            if (sb.length() > 0) {
     36                sb.append('|');
    3637            }
    37             protocolHandlers += pkg;
    38             Utils.updateSystemProperty("java.protocol.handler.pkgs", protocolHandlers);
     38            sb.append(pkg);
     39            Utils.updateSystemProperty("java.protocol.handler.pkgs", sb.toString());
    3940        }
    4041    }
  • trunk/src/org/openstreetmap/josm/tools/ListenerList.java

    r10894 r10936  
    106106    protected void failAdd(T listener) {
    107107        throw new IllegalArgumentException(
    108                 MessageFormat.format("Listener {0} (instance of {1}) was already registered.", listener.toString(),
     108                MessageFormat.format("Listener {0} (instance of {1}) was already registered.", listener,
    109109                        listener.getClass().getName()));
    110110    }
     
    128128        throw new IllegalArgumentException(
    129129                MessageFormat.format("Listener {0} (instance of {1}) was not registered before or already removed.",
    130                         listener.toString(), listener.getClass().getName()));
     130                        listener, listener.getClass().getName()));
    131131    }
    132132
Note: See TracChangeset for help on using the changeset viewer.