Changeset 36072 in osm for applications


Ignore:
Timestamp:
2023-04-19T16:29:43+02:00 (21 months ago)
Author:
taylor.smock
Message:

Cleanup some compile time, checkstyle, and lint warnings

Location:
applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core
Files:
16 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/actions/ViewLicenseAction.java

    r30723 r36072  
    1313import org.openstreetmap.josm.plugins.opendata.core.util.OdUtils;
    1414import org.openstreetmap.josm.tools.CheckParameterUtil;
     15import org.openstreetmap.josm.tools.Logging;
    1516
    1617public class ViewLicenseAction extends JosmAction {
    … …  
    3031            new ViewLicenseDialog(license).showDialog();
    3132        } catch (IOException ex) {
    32             ex.printStackTrace();
     33            Logging.error(ex);
    3334        }
    3435    }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/AbstractDataSetHandler.java

    r36024 r36072  
    8080    private ImageIcon menuIcon;
    8181
    82     public AbstractDataSetHandler() {
     82    protected AbstractDataSetHandler() {
    8383        setShpHandler(new DefaultShpHandler());
    8484        setArchiveHandler(new DefaultArchiveHandler());
    … …  
    9090    private boolean acceptsFilename(String filename, String[] expected, String ... extensions) {
    9191        if (filename != null) {
    92             for (String name : expected) {
     92            for (String expectedName : expected) {
    9393                for (String ext : extensions) {
    94                     if (Pattern.compile(name+"\\."+ext, Pattern.CASE_INSENSITIVE).matcher(filename).matches()) {
    95                         //if (filename.equalsIgnoreCase(name+"."+ext)) {
     94                    if (Pattern.compile(expectedName+"\\."+ext, Pattern.CASE_INSENSITIVE).matcher(filename).matches()) {
     95                        //if (filename.equalsIgnoreCase(expectedName+"."+ext)) {
    9696                        return true;
    9797                    }
    … …  
    102102    }
    103103
    104     protected final boolean acceptsCsvFilename(String filename, String ... expected) {
     104    protected final boolean acceptsCsvFilename(String filename, String... expected) {
    105105        return acceptsFilename(filename, expected, OdConstants.CSV_EXT);
    106106    }
    107107
    108     protected final boolean acceptsXlsFilename(String filename, String ... expected) {
     108    protected final boolean acceptsXlsFilename(String filename, String... expected) {
    109109        return acceptsFilename(filename, expected, OdConstants.XLS_EXT);
    110110    }
    111111
    112     protected final boolean acceptsOdsFilename(String filename, String ... expected) {
     112    protected final boolean acceptsOdsFilename(String filename, String... expected) {
    113113        return acceptsFilename(filename, expected, OdConstants.ODS_EXT);
    114114    }
    115115
    116     protected final boolean acceptsShpFilename(String filename, String ... expected) {
     116    protected final boolean acceptsShpFilename(String filename, String... expected) {
    117117        return acceptsFilename(filename, expected, OdConstants.SHP_EXT);
    118118    }
    119119
    120     protected final boolean acceptsMifFilename(String filename, String ... expected) {
     120    protected final boolean acceptsMifFilename(String filename, String... expected) {
    121121        return acceptsFilename(filename, expected, OdConstants.MIF_EXT);
    122122    }
    123123
    124     protected final boolean acceptsMifTabFilename(String filename, String ... expected) {
     124    protected final boolean acceptsMifTabFilename(String filename, String... expected) {
    125125        return acceptsFilename(filename, expected, OdConstants.MIF_EXT, OdConstants.TAB_EXT);
    126126    }
    127127
    128     protected final boolean acceptsShpMifFilename(String filename, String ... expected) {
     128    protected final boolean acceptsShpMifFilename(String filename, String... expected) {
    129129        return acceptsFilename(filename, expected, OdConstants.SHP_EXT, OdConstants.MIF_EXT);
    130130    }
    131131
    132     protected final boolean acceptsKmlFilename(String filename, String ... expected) {
     132    protected final boolean acceptsKmlFilename(String filename, String... expected) {
    133133        return acceptsFilename(filename, expected, OdConstants.KML_EXT);
    134134    }
    135135
    136     protected final boolean acceptsKmzFilename(String filename, String ... expected) {
     136    protected final boolean acceptsKmzFilename(String filename, String... expected) {
    137137        return acceptsFilename(filename, expected, OdConstants.KMZ_EXT);
    138138    }
    139139
    140     protected final boolean acceptsKmzShpFilename(String filename, String ... expected) {
     140    protected final boolean acceptsKmzShpFilename(String filename, String... expected) {
    141141        return acceptsFilename(filename, expected, OdConstants.KMZ_EXT, OdConstants.SHP_EXT);
    142142    }
    143143
    144     protected final boolean acceptsKmzTabFilename(String filename, String ... expected) {
     144    protected final boolean acceptsKmzTabFilename(String filename, String... expected) {
    145145        return acceptsFilename(filename, expected, OdConstants.KMZ_EXT, OdConstants.TAB_EXT);
    146146    }
    147147
    148     protected final boolean acceptsZipFilename(String filename, String ... expected) {
     148    protected final boolean acceptsZipFilename(String filename, String... expected) {
    149149        return acceptsFilename(filename, expected, OdConstants.ZIP_EXT);
    150150    }
    151151
    152     protected final boolean accepts7ZipFilename(String filename, String ... expected) {
     152    protected final boolean accepts7ZipFilename(String filename, String... expected) {
    153153        return acceptsFilename(filename, expected, OdConstants.SEVENZIP_EXT);
    154154    }
    155155
    156     protected final boolean acceptsCsvKmzFilename(String filename, String ... expected) {
     156    protected final boolean acceptsCsvKmzFilename(String filename, String... expected) {
    157157        return acceptsFilename(filename, expected, OdConstants.CSV_EXT, OdConstants.KMZ_EXT);
    158158    }
    159159
    160     protected final boolean acceptsCsvKmzTabFilename(String filename, String ... expected) {
     160    protected final boolean acceptsCsvKmzTabFilename(String filename, String... expected) {
    161161        return acceptsFilename(filename, expected, OdConstants.CSV_EXT, OdConstants.KMZ_EXT, OdConstants.TAB_EXT);
    162162    }
    163163
    164     protected final boolean acceptsCsvXlsFilename(String filename, String ... expected) {
     164    protected final boolean acceptsCsvXlsFilename(String filename, String... expected) {
    165165        return acceptsFilename(filename, expected, OdConstants.CSV_EXT, OdConstants.XLS_EXT);
    166166    }
    … …  
    544544
    545545    public final boolean addTool(JosmAction tool) {
    546         return tool != null ? tools.add(tool) : false;
     546        return tool != null && tools.add(tool);
    547547    }
    548548
    549549    public final boolean removeTool(JosmAction tool) {
    550         return tool != null ? tools.remove(tool) : false;
     550        return tool != null && tools.remove(tool);
    551551    }
    552552

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/SimpleDataSetHandler.java

    r35899 r36072  
    3333    private final boolean relevantUnion;
    3434
    35     public SimpleDataSetHandler() {
     35    protected SimpleDataSetHandler() {
    3636        this.relevantUnion = false;
    3737    }
    3838
    39     public SimpleDataSetHandler(String relevantTag) {
     39    protected SimpleDataSetHandler(String relevantTag) {
    4040        addRelevantTag(relevantTag);
    4141        this.relevantUnion = false;
    … …  
    5151    }
    5252
    53     public SimpleDataSetHandler(boolean relevantUnion, String... relevantTags) {
     53    protected SimpleDataSetHandler(boolean relevantUnion, String... relevantTags) {
    5454        addRelevantTag(relevantTags);
    5555        this.relevantUnion = relevantUnion;
    5656    }
    5757
    58     public SimpleDataSetHandler(String... relevantTags) {
     58    protected SimpleDataSetHandler(String... relevantTags) {
    5959        this(false, relevantTags);
    6060    }
    6161
    62     public SimpleDataSetHandler(Tag relevantTag) {
     62    protected SimpleDataSetHandler(Tag relevantTag) {
    6363        addRelevantTag(relevantTag);
    6464        this.relevantUnion = false;
    6565    }
    6666
    67     public SimpleDataSetHandler(boolean relevantUnion, Tag ... relevantTags) {
     67    protected SimpleDataSetHandler(boolean relevantUnion, Tag ... relevantTags) {
    6868        addRelevantTag(relevantTags);
    6969        this.relevantUnion = relevantUnion;
    7070    }
    7171
    72     public SimpleDataSetHandler(Tag ... relevantTags) {
     72    protected SimpleDataSetHandler(Tag... relevantTags) {
    7373        this(false, relevantTags);
    7474    }
    7575
    76     public void addRelevantTag(String ... relevantTags) {
     76    public void addRelevantTag(String... relevantTags) {
    7777        addTags(this.relevantTags, relevantTags);
    7878    }
    7979
    80     public void addRelevantTag(Tag ... relevantTags) {
     80    public void addRelevantTag(Tag... relevantTags) {
    8181        addTags(this.relevantTags, relevantTags);
    8282    }
    8383
    84     public void addForbiddenTag(String ... forbiddenTags) {
     84    public void addForbiddenTag(String... forbiddenTags) {
    8585        addTags(this.forbiddenTags, forbiddenTags);
    8686    }
    8787
    88     public void addForbiddenTag(Tag ... forbiddenTags) {
     88    public void addForbiddenTag(Tag... forbiddenTags) {
    8989        addTags(this.forbiddenTags, forbiddenTags);
    9090    }
    9191
    92     private void addTags(final List<Tag> list, String ... tags) {
     92    private static void addTags(final List<Tag> list, String... tags) {
    9393        if (tags != null) {
    9494            for (String tag : tags) {
    … …  
    105105    }
    106106
    107     private void addTags(final List<Tag> list, Tag ... tags) {
     107    private static void addTags(final List<Tag> list, Tag... tags) {
    108108        if (tags != null) {
    109109            for (Tag tag : tags) {
    … …  
    133133            }
    134134        }
    135         if (isForbidden(p)) {
    136             return false;
    137         }
    138         return true;
     135        return !isForbidden(p);
    139136    }
    140137
    … …  
    166163        String[] mpconditions = new String[conditions.length+1];
    167164        mpconditions[0] = OverpassApi.hasKey("type", "multipolygon");
    168         for (int i = 0; i < conditions.length; i++) {
    169             mpconditions[i+1] = conditions[i];
    170         }
     165        System.arraycopy(conditions, 0, mpconditions, 1, conditions.length);
    171166        return OverpassApi.query(bbox, NODE, conditions) + "\n" + // Nodes
    172167        OverpassApi.recurse(NODE_RELATION, RELATION_WAY, WAY_NODE) + "\n" +
    … …  
    194189    @Override
    195190    protected Collection<String> getOsmXapiRequests(String bbox) {
    196         String relevantTags = "";
    197         for (Tag tag : this.relevantTags) {
    198             relevantTags += "["+tag.getKey()+"="+(tag.getValue() == null ? "*" : tag.getValue())+"]";
    199         }
    200         String forbiddenTags = "";
     191        StringBuilder relevantTagsSB = new StringBuilder();
     192        for (Tag tag : this.relevantTags) {
     193            relevantTagsSB.append("[").append(tag.getKey()).append("=").append(tag.getValue() == null ? "*" : tag.getValue()).append("]");
     194        }
     195        StringBuilder forbiddenTagsSB = new StringBuilder();
    201196        for (Tag tag : this.forbiddenTags) {
    202             forbiddenTags += "[not("+tag.getKey()+"="+(tag.getValue() == null ? "*" : tag.getValue())+")]";
    203         }
    204         return Collections.singleton("*[bbox="+bbox+"]"+relevantTags+forbiddenTags+"[@meta]");
     197            forbiddenTagsSB.append("[not(").append(tag.getKey()).append("=").append(tag.getValue() == null ? "*" : tag.getValue()).append(")]");
     198        }
     199        return Collections.singleton("*[bbox="+bbox+"]"+relevantTagsSB+forbiddenTagsSB+"[@meta]");
    205200    }
    206201}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/WayCombiner.java

    r34662 r36072  
    3333    }
    3434
    35     protected static Way getTargetWay(Collection<Way> combinedWays) {
     35    private static Way getTargetWay(Collection<Way> combinedWays) {
    3636        // init with an arbitrary way
    3737        Way targetWay = combinedWays.iterator().next();

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/be/BelgianDataSetHandler.java

    r34452 r36072  
    1212import org.openstreetmap.josm.plugins.opendata.core.OdConstants;
    1313import org.openstreetmap.josm.plugins.opendata.core.datasets.SimpleDataSetHandler;
     14import org.openstreetmap.josm.plugins.opendata.core.io.tabular.CsvHandler;
    1415import org.openstreetmap.josm.plugins.opendata.core.io.tabular.DefaultCsvHandler;
    1516import org.openstreetmap.josm.spi.preferences.Config;
     17import org.openstreetmap.josm.tools.Logging;
    1618
    1719public abstract class BelgianDataSetHandler extends SimpleDataSetHandler implements BelgianConstants {
    … …  
    5254    }
    5355
    54     public BelgianDataSetHandler() {
     56    protected BelgianDataSetHandler() {
    5557        init();
    5658    }
    5759
    58     public BelgianDataSetHandler(String relevantTag) {
     60    protected BelgianDataSetHandler(String relevantTag) {
    5961        super(relevantTag);
    6062        init();
    6163    }
    6264
    63     public BelgianDataSetHandler(boolean relevantUnion, String[] relevantTags) {
     65    protected BelgianDataSetHandler(boolean relevantUnion, String[] relevantTags) {
    6466        super(relevantUnion, relevantTags);
    6567        init();
    6668    }
    6769
    68     public BelgianDataSetHandler(boolean relevantUnion, Tag[] relevantTags) {
     70    protected BelgianDataSetHandler(boolean relevantUnion, Tag[] relevantTags) {
    6971        super(relevantUnion, relevantTags);
    7072        init();
    … …  
    8587    protected final void setSingleProjection(Projection singleProjection) {
    8688        this.singleProjection = singleProjection;
    87         getCsvHandler().setHandlesProjection(singleProjection != null);
     89        final CsvHandler handler = getCsvHandler();
     90        if (handler != null) {
     91            handler.setHandlesProjection(singleProjection != null);
     92        }
    8893    }
    8994
    … …  
    108113            return new URL(BELGIAN_PORTAL.replace(OdConstants.PATTERN_LANG, lang.substring(0, 2))+nationalPortalPath); //FIXME
    109114        } catch (MalformedURLException e) {
    110             e.printStackTrace();
     115            Logging.error(e);
    111116        }
    112117        return null;

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/fr/FrenchDataSetHandler.java

    r32545 r36072  
    1818import org.openstreetmap.josm.plugins.opendata.core.datasets.SimpleDataSetHandler;
    1919import org.openstreetmap.josm.plugins.opendata.core.io.tabular.DefaultCsvHandler;
     20import org.openstreetmap.josm.tools.Logging;
    2021
    2122public abstract class FrenchDataSetHandler extends SimpleDataSetHandler implements FrenchConstants {
    … …  
    7172    }
    7273
    73     public FrenchDataSetHandler() {
    74         init();
    75     }
    76 
    77     public FrenchDataSetHandler(String relevantTag) {
     74    protected FrenchDataSetHandler() {
     75        init();
     76    }
     77
     78    protected FrenchDataSetHandler(String relevantTag) {
    7879        super(relevantTag);
    7980        init();
    8081    }
    8182
    82     public FrenchDataSetHandler(boolean relevantUnion, String[] relevantTags) {
     83    protected FrenchDataSetHandler(boolean relevantUnion, String[] relevantTags) {
    8384        super(relevantUnion, relevantTags);
    8485        init();
    8586    }
    8687
    87     public FrenchDataSetHandler(boolean relevantUnion, Tag[] relevantTags) {
     88    protected FrenchDataSetHandler(boolean relevantUnion, Tag[] relevantTags) {
    8889        super(relevantUnion, relevantTags);
    8990        init();
    … …  
    110111            }
    111112        } catch (MalformedURLException e) {
    112             e.printStackTrace();
     113            Logging.error(e);
    113114        }
    114115        return null;
    … …  
    125126    }
    126127
    127     protected static final LatLon getLatLonByDptCode(EastNorth en, String dpt, boolean useCC9) {
     128    protected static LatLon getLatLonByDptCode(EastNorth en, String dpt, boolean useCC9) {
    128129        // CHECKSTYLE.OFF: LineLength
    129130        if (dpt.equals("971") || dpt.equals("972") || dpt.equals("977") || dpt.equals("978")) {    // Antilles
    … …  
    161162    }
    162163
    163     private void replaceFaxPhone(OsmPrimitive p, String dataKey, String osmKey) {
     164    private static void replaceFaxPhone(OsmPrimitive p, String dataKey, String osmKey) {
    164165        String phone = p.get(dataKey);
    165166        if (phone != null) {
    … …  
    204205                        Matcher m2 = Pattern.compile(dayGroup+"(?:"+sep+dayGroup+")+").matcher(range);
    205206                        if (m2.matches()) {
    206                             String replacement = "";
     207                            StringBuilder replacement = new StringBuilder();
    207208                            for (int i = 0; i < m2.groupCount(); i++) {
    208209                                if (i > 0) {
    209                                     replacement += sep;
     210                                    replacement.append(sep);
    210211                                }
    211                                 replacement += getEnDay(m2.group(i+1));
     212                                replacement.append(getEnDay(m2.group(i + 1)));
    212213                            }
    213                             hours = hours.replace(range, replacement);
     214                            hours = hours.replace(range, replacement.toString());
    214215                        }
    215216                    } else {
    … …  
    223224    }
    224225
    225     private String getEnDay(String frDay) {
     226    private static String getEnDay(String frDay) {
    226227        for (int i = 0; i < dayFrSplit.length; i++) {
    227228            if (dayFrSplit[i].equals(frDay)) {

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/DialogPrompter.java

    r32545 r36072  
    22package org.openstreetmap.josm.plugins.opendata.core.gui;
    33
    4 import java.lang.reflect.InvocationTargetException;
    5 
    6 import javax.swing.SwingUtilities;
    7 
    84import org.openstreetmap.josm.gui.ExtendedDialog;
     5import org.openstreetmap.josm.gui.util.GuiHelper;
    96
    107public class DialogPrompter<T extends ExtendedDialog> implements Runnable {
    … …  
    4138
    4239    public final DialogPrompter<T> promptInEdt() {
    43         if (SwingUtilities.isEventDispatchThread()) {
    44             run();
    45         } else {
    46             try {
    47                 SwingUtilities.invokeAndWait(this);
    48             } catch (InterruptedException e) {
    49                 e.printStackTrace();
    50             } catch (InvocationTargetException e) {
    51                 e.printStackTrace();
    52             }
    53         }
     40        GuiHelper.runInEDTAndWait(this);
    5441        return this;
    5542    }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/ModulePreference.java

    r35513 r36072  
    1717import java.util.Collection;
    1818import java.util.Collections;
    19 import java.util.Iterator;
    2019import java.util.LinkedList;
    2120import java.util.List;
    … …  
    101100     * by the user
    102101     */
    103     private boolean modulePreferencesActivated = false;
     102    private boolean modulePreferencesActivated;
    104103
    105104    protected JPanel buildSearchFieldPanel() {
    … …  
    116115        gc.gridx = 1;
    117116        gc.weightx = 1.0;
    118         pnl.add(tfFilter = new JTextField(), gc);
     117        this.tfFilter = new JTextField();
     118        pnl.add(this.tfFilter, gc);
    119119        tfFilter.setToolTipText(tr("Enter a search expression"));
    120120        SelectAllOnFocusGainedDecorator.decorate(tfFilter);
    … …  
    332332                model.refreshLocalModuleVersion(moduleDownloadTask.getDownloadedModules());
    333333                model.clearPendingModules(moduleDownloadTask.getDownloadedModules());
    334                 GuiHelper.runInEDT(() -> pnlModulePreferences.refreshView());
     334                GuiHelper.runInEDT(pnlModulePreferences::refreshView);
    335335            };
    336336
    … …  
    343343                // select modules which actually have to be updated
    344344                //
    345                 Iterator<ModuleInformation> it = toUpdate.iterator();
    346                 while (it.hasNext()) {
    347                     ModuleInformation pi = it.next();
    348                     if (!pi.isUpdateRequired()) {
    349                         it.remove();
    350                     }
    351                 }
     345                toUpdate.removeIf(pi -> !pi.isUpdateRequired());
    352346                if (toUpdate.isEmpty()) {
    353347                    alertNothingToUpdate();
    … …  
    387381     */
    388382    class ModulePreferenceActivationListener implements ChangeListener {
    389         private Component pane;
     383        private final Component pane;
    390384        ModulePreferenceActivationListener(Component preferencesPane) {
    391385            pane = preferencesPane;
    … …  
    409403        public void filter() {
    410404            String expr = tfFilter.getText().trim();
    411             if (expr.equals("")) {
     405            if ("".equals(expr)) {
    412406                expr = null;
    413407            }
    … …  
    514508                ret.add(model.get(i));
    515509            }
    516             return ret;
     510            return Collections.unmodifiableList(ret);
    517511        }
    518512    }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/ModulePreferencesModel.java

    r34452 r36072  
    160160        return displayedModules;
    161161    }
    162 
    163162
    164163    /**

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/OdDialog.java

    r34116 r36072  
    5757    }
    5858
    59     private class SelectAction extends JosmAction {
     59    private static class SelectAction extends JosmAction {
    6060        SelectAction() {
    6161            super(marktr("Select"), "dialogs/select",
    … …  
    6969    }
    7070
    71     private class DiffAction extends JosmAction {
     71    private static class DiffAction extends JosmAction {
    7272        DiffAction() {
    7373            super(marktr("Diff"), "dialogs/diff", tr("Perform diff between current data set and existing OSM data."), null, false);
    … …  
    8080    }
    8181
    82     private class ToolsAction extends JosmAction {
     82    private static class ToolsAction extends JosmAction {
    8383        ToolsAction() {
    8484            super(marktr("Tools"), "dialogs/utils", tr("Open tools menu for this data."), null, false);
    … …  
    9696                        KeyEvent.VK_A, Shortcut.ALT_CTRL_SHIFT), 150, false, OdPreferenceSetting.class);
    9797
    98         this.buttons = Arrays.asList(new SideButton[] {
    99                 /*selectButton =*/ new SideButton(new SelectAction()),
    100                 downloadButton = new SideButton(new DownloadAction()),
    101                 diffButton = new SideButton(new DiffAction()),
    102                 toolsButton = new SideButton(new ToolsAction())
    103         });
     98        this.downloadButton = new SideButton(new DownloadAction());
     99        this.diffButton = new SideButton(new DiffAction());
     100        this.toolsButton = new SideButton(new ToolsAction());
     101        this.buttons = Arrays.asList(/*selectButton =*/ new SideButton(new SelectAction()),
     102                this.downloadButton, this.diffButton, this.toolsButton);
     103
    104104
    105105        this.toolsButton.addMouseListener(new MouseAdapter() {

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/ViewLicenseDialog.java

    r34452 r36072  
    99import java.io.IOException;
    1010
    11 import javax.swing.Icon;
    1211import javax.swing.JEditorPane;
    1312import javax.swing.JOptionPane;
    … …  
    1817import org.openstreetmap.josm.plugins.opendata.core.licenses.License;
    1918import org.openstreetmap.josm.tools.ImageProvider;
     19import org.openstreetmap.josm.tools.Logging;
    2020
    2121public class ViewLicenseDialog extends ExtendedDialog {
    … …  
    4545        scrollPane.setPreferredSize(new Dimension(800, 600));
    4646
    47         setButtonIcons(new Icon[] {
    48                 ImageProvider.get("ok"),
     47        setButtonIcons(ImageProvider.get("ok"),
    4948                ImageProvider.get("agreement24"),
    50                 ImageProvider.get("cancel"),
    51                 });
    52         setToolTipTexts(new String[] {
    53                 null,
     49                ImageProvider.get("cancel"));
     50        setToolTipTexts(null,
    5451                tr("View the full text of this license"),
    55                 null});
     52                null);
    5653        if (license.getIcon() != null) {
    5754            setIcon(license.getIcon());
    … …  
    7673                }
    7774            } catch (IOException e) {
    78                 e.printStackTrace();
     75                Logging.error(e);
    7976            }
    8077            summary = !summary;

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/OsmDownloader.java

    r34452 r36072  
    2525                MainApplication.getMenu().openLocation.openUrl(false, oapiServer+"data="+oapiReqEnc);
    2626            } catch (UnsupportedEncodingException e) {
    27                 e.printStackTrace();
     27                Logging.error(e);
    2828            }
    2929        }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/archive/ArchiveReader.java

    r33518 r36072  
    4848    private File file;
    4949
    50     public ArchiveReader(AbstractDataSetHandler handler, ArchiveHandler archiveHandler, boolean promptUser) {
     50    protected ArchiveReader(AbstractDataSetHandler handler, ArchiveHandler archiveHandler, boolean promptUser) {
    5151        this.handler = handler;
    5252        this.archiveHandler = archiveHandler;
    … …  
    123123            }
    124124        } catch (IllegalArgumentException e) {
    125             Logging.error(e.getMessage());
     125            Logging.error(e);
    126126        } finally {
    127127            OdUtils.deleteDir(temp);
    … …  
    149149            Logging.info("Parsing file "+f.getName());
    150150            DataSet from = null;
    151             FileInputStream in = new FileInputStream(f);
    152             ProgressMonitor instance = null;
    153             if (progressMonitor != null) {
    154                 instance = progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false);
    155             }
    156             if (f.getName().toLowerCase().endsWith(OdConstants.CSV_EXT)) {
    157                 from = CsvReader.parseDataSet(in, handler, instance);
    158             } else if (f.getName().toLowerCase().endsWith(OdConstants.KML_EXT)) {
    159                 from = KmlReader.parseDataSet(in, instance);
    160             } else if (f.getName().toLowerCase().endsWith(OdConstants.KMZ_EXT)) {
    161                 from = KmzReader.parseDataSet(in, instance);
    162             } else if (f.getName().toLowerCase().endsWith(OdConstants.XLS_EXT)) {
    163                 from = XlsReader.parseDataSet(in, handler, instance);
    164             } else if (f.getName().toLowerCase().endsWith(OdConstants.ODS_EXT)) {
    165                 from = OdsReader.parseDataSet(in, handler, instance);
    166             } else if (f.getName().toLowerCase().endsWith(OdConstants.SHP_EXT)) {
    167                 from = ShpReader.parseDataSet(in, f, handler, instance);
    168             } else if (f.getName().toLowerCase().endsWith(OdConstants.MIF_EXT)) {
    169                 from = MifReader.parseDataSet(in, f, handler, instance);
    170             } else if (f.getName().toLowerCase().endsWith(OdConstants.TAB_EXT)) {
    171                 from = TabReader.parseDataSet(in, f, handler, instance);
    172             } else if (f.getName().toLowerCase().endsWith(OdConstants.GML_EXT)) {
    173                 from = GmlReader.parseDataSet(in, handler, instance);
    174             } else if (f.getName().toLowerCase().endsWith(OdConstants.XML_EXT)) {
    175                 if (OdPlugin.getInstance().xmlImporter.acceptFile(f)) {
    176                     from = NeptuneReader.parseDataSet(in, handler, instance);
     151            try (FileInputStream in = new FileInputStream(f)) {
     152                ProgressMonitor instance = null;
     153                if (progressMonitor != null) {
     154                    instance = progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false);
     155                }
     156                final String lowerCaseName = f.getName().toLowerCase();
     157                if (lowerCaseName.endsWith(OdConstants.CSV_EXT)) {
     158                    from = CsvReader.parseDataSet(in, handler, instance);
     159                } else if (lowerCaseName.endsWith(OdConstants.KML_EXT)) {
     160                    from = KmlReader.parseDataSet(in, instance);
     161                } else if (lowerCaseName.endsWith(OdConstants.KMZ_EXT)) {
     162                    from = KmzReader.parseDataSet(in, instance);
     163                } else if (lowerCaseName.endsWith(OdConstants.XLS_EXT)) {
     164                    from = XlsReader.parseDataSet(in, handler, instance);
     165                } else if (lowerCaseName.endsWith(OdConstants.ODS_EXT)) {
     166                    from = OdsReader.parseDataSet(in, handler, instance);
     167                } else if (lowerCaseName.endsWith(OdConstants.SHP_EXT)) {
     168                    from = ShpReader.parseDataSet(in, f, handler, instance);
     169                } else if (lowerCaseName.endsWith(OdConstants.MIF_EXT)) {
     170                    from = MifReader.parseDataSet(in, f, handler, instance);
     171                } else if (lowerCaseName.endsWith(OdConstants.TAB_EXT)) {
     172                    from = TabReader.parseDataSet(in, f, handler, instance);
     173                } else if (lowerCaseName.endsWith(OdConstants.GML_EXT)) {
     174                    from = GmlReader.parseDataSet(in, handler, instance);
     175                } else if (lowerCaseName.endsWith(OdConstants.XML_EXT)) {
     176                    if (OdPlugin.getInstance().xmlImporter.acceptFile(f)) {
     177                        from = NeptuneReader.parseDataSet(in, handler, instance);
     178                    } else {
     179                        Logging.warn("Unsupported XML file: " + f.getName());
     180                    }
    177181                } else {
    178                     Logging.warn("Unsupported XML file: "+f.getName());
    179                 }
    180             } else {
    181                 Logging.warn("Unsupported file extension: "+f.getName());
    182             }
    183             return from;
     182                    Logging.warn("Unsupported file extension: " + f.getName());
     183                }
     184                return from;
     185            }
    184186        }
    185187    }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/archive/CandidateChooser.java

    r32545 r36072  
    3333    }
    3434
    35     private class Renderer extends DefaultListCellRenderer {
     35    private static class Renderer extends DefaultListCellRenderer {
    3636
    3737        @Override

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic/GeographicReader.java

    r36024 r36072  
    2222import javax.json.JsonReader;
    2323import javax.json.spi.JsonProvider;
    24 import javax.swing.Icon;
    2524import javax.swing.JOptionPane;
    2625
    … …  
    3938import org.opengis.referencing.FactoryException;
    4039import org.opengis.referencing.IdentifiedObject;
    41 import org.opengis.referencing.NoSuchAuthorityCodeException;
    4240import org.opengis.referencing.crs.CoordinateReferenceSystem;
    4341import org.opengis.referencing.crs.ProjectedCRS;
    … …  
    8078        try {
    8179            wgs84 = CRS.decode("EPSG:4326");
    82         } catch (NoSuchAuthorityCodeException e) {
    83             e.printStackTrace();
    8480        } catch (FactoryException e) {
    85             e.printStackTrace();
     81            Logging.error(e);
    8682        }
    8783    }
    … …  
    9793    private static final Map<String, Integer> esriWkid = new TreeMap<>();
    9894
    99     public GeographicReader(GeographicHandler handler, GeographicHandler[] defaultHandlers) {
     95    protected GeographicReader(GeographicHandler handler, GeographicHandler[] defaultHandlers) {
    10096        this.nodes = new HashMap<>();
    10197        this.handler = handler;
    … …  
    225221    }
    226222
    227     protected final void addWayToMp(Relation r, String role, Way w) {
     223    protected static void addWayToMp(Relation r, String role, Way w) {
    228224        r.addMember(new RelationMember(role, w));
    229225    }
    … …  
    233229     * want to continue
    234230     */
    235     protected static final boolean warnLenientMethod(final Component parent, final CoordinateReferenceSystem crs) {
     231    protected static boolean warnLenientMethod(final Component parent, final CoordinateReferenceSystem crs) {
    236232        return new DialogPrompter<ExtendedDialog>() {
    237233            @Override
    … …  
    239235                final ExtendedDialog dlg = new ExtendedDialog(parent,
    240236                        tr("Cannot transform to WGS84"),
    241                         new String[] {tr("Cancel"), tr("Continue")});
     237                        tr("Cancel"), tr("Continue"));
    242238                // CHECKSTYLE.OFF: LineLength
    243239                dlg.setContent("<html>" +
    … …  
    247243                        "</html>");
    248244                // CHECKSTYLE.ON: LineLength
    249                 dlg.setButtonIcons(new Icon[] {
    250                         new ImageProvider("cancel").setMaxSize(ImageSizes.LARGEICON).get(),
     245                dlg.setButtonIcons(new ImageProvider("cancel").setMaxSize(ImageSizes.LARGEICON).get(),
    251246                        new ImageProvider("ok").setMaxSize(ImageSizes.LARGEICON).addOverlay(
    252                                 new ImageOverlay(new ImageProvider("warning-small"), 0.5, 0.5, 1.0, 1.0)).get()});
    253                 dlg.setToolTipTexts(new String[] {
    254                         tr("Cancel"),
    255                         tr("Try lenient method")});
     247                                new ImageOverlay(new ImageProvider("warning-small"), 0.5, 0.5, 1.0, 1.0)).get());
     248                dlg.setToolTipTexts(tr("Cancel"),
     249                        tr("Try lenient method"));
    256250                dlg.setIcon(JOptionPane.WARNING_MESSAGE);
    257251                dlg.setCancelButton(1);

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic/GmlReader.java

    r35125 r36072  
    99import java.util.Map;
    1010
    11 import javax.xml.stream.XMLInputFactory;
    1211import javax.xml.stream.XMLStreamConstants;
    1312import javax.xml.stream.XMLStreamException;
    … …  
    3534import org.openstreetmap.josm.tools.Logging;
    3635import org.openstreetmap.josm.tools.UserCancelException;
     36import org.openstreetmap.josm.tools.XmlUtils;
    3737
    3838/**
    … …  
    5959    private int dim;
    6060
    61     private final class CrsData {
     61    private static final class CrsData {
    6262        public CoordinateReferenceSystem crs;
    6363        public MathTransform transform;
    … …  
    8181            throws IOException, XMLStreamException {
    8282        InputStreamReader ir = UTFInputStreamReader.create(in, OdConstants.UTF8);
    83         XMLStreamReader parser = XMLInputFactory.newInstance().createXMLStreamReader(ir);
     83        XMLStreamReader parser = XmlUtils.newSafeXMLInputFactory().createXMLStreamReader(ir);
    8484        try {
    8585            return new GmlReader(parser, handler != null ? handler.getGmlHandler() : null).parseDoc(instance);
    … …  
    9898        while (parser.hasNext()) {
    9999            int event = parser.next();
    100             if (event == XMLStreamConstants.START_ELEMENT) {
    101                 if (isElement(GML_FEATURE_MEMBER)) {
    102                     try {
    103                         parseFeatureMember(parent);
    104                     } catch (UserCancelException e) {
    105                         return ds;
    106                     }
     100            if (event == XMLStreamConstants.START_ELEMENT && isElement(GML_FEATURE_MEMBER)) {
     101                try {
     102                    parseFeatureMember(parent);
     103                } catch (UserCancelException e) {
     104                    Logging.trace(e);
     105                    return ds;
    107106                }
    108107            }
    … …  
    132131            try {
    133132                findCRS(srs);
    134             } catch (NoSuchAuthorityCodeException e) {
    135                 e.printStackTrace();
    136133            } catch (FactoryException e) {
    137                 e.printStackTrace();
     134                Logging.error(e);
    138135            }
    139136            if (crs == null) {
    … …  
    172169                    String[] tab = parser.getElementText().split(" ");
    173170                    for (int i = 0; i < tab.length; i += dim) {
    174                         Point p = geometryFactory.createPoint(new Coordinate(Double.valueOf(tab[i]), Double.valueOf(tab[i+1])));
    175                         node = createOrGetNode(p, dim > 2 && !tab[i+2].equals("0") ? tab[i+2] : null);
     171                        Point p = geometryFactory.createPoint(new Coordinate(Double.parseDouble(tab[i]), Double.parseDouble(tab[i+1])));
     172                        node = createOrGetNode(p, dim > 2 && !"0".equals(tab[i+2]) ? tab[i+2] : null);
    176173                        if (way != null) {
    177174                            way.addNode(node);
    … …  
    182179                } else if (isElement(GML_COORDINATES)) {
    183180                    String[] tab = parser.getElementText().trim().split(",");
    184                     Point p = geometryFactory.createPoint(new Coordinate(Double.valueOf(tab[0]), Double.valueOf(tab[1])));
     181                    Point p = geometryFactory.createPoint(new Coordinate(Double.parseDouble(tab[0]), Double.parseDouble(tab[1])));
    185182                    node = createOrGetNode(p);
    186183                    if (way == null) {
    … …  
    207204        }
    208205        if (prim != null) {
    209             for (String k : tags.keySet()) {
    210                 prim.put(k, tags.get(k).toString());
     206            for (Map.Entry<String, StringBuilder> entry : tags.entrySet()) {
     207                prim.put(entry.getKey(), entry.getValue().toString());
    211208            }
    212209        }
Note: See TracChangeset for help on using the changeset viewer.