Changeset 7029 in josm for trunk


Ignore:
Timestamp:
2014-04-29T16:38:07+02:00 (11 years ago)
Author:
Don-vip
Message:

fix more warnings, remove unused code

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/data/AutosaveTask.java

    r7005 r7029  
    179179    }
    180180
    181     private void savelayer(AutosaveLayerInfo info) throws IOException {
     181    private void savelayer(AutosaveLayerInfo info) {
    182182        if (!info.layer.getName().equals(info.layerName)) {
    183183            setLayerFileName(info);
    … …  
    263263                    if (info.layer == osmLayer) {
    264264
    265                         try {
    266                             savelayer(info);
    267                             File lastFile = info.backupFiles.pollLast();
    268                             if (lastFile != null) {
    269                                 moveToDeletedLayersFolder(lastFile);
     265                        savelayer(info);
     266                        File lastFile = info.backupFiles.pollLast();
     267                        if (lastFile != null) {
     268                            moveToDeletedLayersFolder(lastFile);
     269                        }
     270                        for (File file: info.backupFiles) {
     271                            if (file.delete()) {
     272                                getPidFile(file).delete();
    270273                            }
    271                             for (File file: info.backupFiles) {
    272                                 if (file.delete()) {
    273                                     getPidFile(file).delete();
    274                                 }
    275                             }
    276                         } catch (IOException e) {
    277                             Main.error(tr("Error while creating backup of removed layer: {0}", e.getMessage()));
    278274                        }
    279275

  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListTableCellRenderer.java

    r7018 r7029  
    131131    @SuppressWarnings("unchecked")
    132132    protected ListMergeModel<Node>.EntriesTableModel getModel(JTable table) {
    133         return (ListMergeModel.EntriesTableModel)table.getModel();
     133        return (ListMergeModel<Node>.EntriesTableModel)table.getModel();
    134134    }
    135135}

  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/relation/RelationMemberTableCellRenderer.java

    r7018 r7029  
    147147    @SuppressWarnings("unchecked")
    148148    protected ListMergeModel<Node>.EntriesTableModel getModel(JTable table) {
    149         return (ListMergeModel.EntriesTableModel)table.getModel();
     149        return (ListMergeModel<Node>.EntriesTableModel)table.getModel();
    150150    }
    151151}

  • trunk/src/org/openstreetmap/josm/gui/dialogs/RelationListDialog.java

    r7021 r7029  
    563563        }
    564564
    565         /**
    566          * Returns the index of the relation
    567          * @param rel The relation to look for
    568          *
    569          * @return index of relation (null if it cannot be found)
    570          */
    571         public Integer getRelationIndex(Relation rel) {
    572             int i = relations.indexOf(rel);
    573             if (i<0)
    574                 return null;
    575             return i;
    576         }
    577 
    578565        private Integer getVisibleRelationIndex(Relation rel) {
    579566            int i = getVisibleRelations().indexOf(rel);
    … …  
    631618    }
    632619
     620    /**
     621     * Replies the list of selected relations. Empty list, if there are no selected relations.
     622     * @return the list of selected, non-new relations.
     623     */
    633624    public Collection<Relation> getSelectedRelations() {
    634625        return model.getSelectedRelations();

  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/TagEditHelper.java

    r7005 r7029  
    5353import javax.swing.JPopupMenu;
    5454import javax.swing.KeyStroke;
     55import javax.swing.ListCellRenderer;
    5556import javax.swing.table.DefaultTableModel;
    5657import javax.swing.text.JTextComponent;
    … …  
    8788    // Selection that we are editing by using both dialogs
    8889    Collection<OsmPrimitive> sel;
    89    
     90
    9091    private String changedKey;
    9192    private String objKey;
    … …  
    127128
    128129        final AddTagsDialog addDialog = new AddTagsDialog();
    129        
     130
    130131        addDialog.showDialog();
    131        
     132
    132133        addDialog.destroyActions();
    133134        if (addDialog.getValue() == 1)
    … …  
    136137            addDialog.undoAllTagsAdding();
    137138    }
    138    
     139
    139140    /**
    140141    * Edit the value in the tags table row
    … …  
    150151        String key = tagData.getValueAt(row, 0).toString();
    151152        objKey=key;
    152        
     153
    153154        @SuppressWarnings("unchecked")
    154155        final EditTagDialog editDialog = new EditTagDialog(key, row,
    … …  
    158159        editDialog.performTagEdit();
    159160    }
    160    
     161
    161162    /**
    162163     * If during last editProperty call user changed the key name, this key will be returned
    … …  
    166167        return changedKey;
    167168    }
    168    
     169
    169170    public void resetChangedKey() {
    170171        changedKey = null;
    … …  
    233234                }
    234235            };
    235        
    236         DefaultListCellRenderer cellRenderer = new DefaultListCellRenderer() {
    237                 @Override public Component getListCellRendererComponent(JList list,
    238                         Object value, int index, boolean isSelected,  boolean cellHasFocus){
    239                     Component c = super.getListCellRendererComponent(list, value,
    240                             index, isSelected, cellHasFocus);
     236
     237        ListCellRenderer<AutoCompletionListItem> cellRenderer = new ListCellRenderer<AutoCompletionListItem>() {
     238            final DefaultListCellRenderer def = new DefaultListCellRenderer();
     239                @Override
     240                public Component getListCellRendererComponent(JList<? extends AutoCompletionListItem> list,
     241                        AutoCompletionListItem value, int index, boolean isSelected,  boolean cellHasFocus){
     242                    Component c = def.getListCellRendererComponent(list, value, index, isSelected, cellHasFocus);
    241243                    if (c instanceof JLabel) {
    242                         String str = ((AutoCompletionListItem) value).getValue();
     244                        String str = value.getValue();
    243245                        if (valueCount.containsKey(objKey)) {
    244246                            Map<String, Integer> m = valueCount.get(objKey);
    … …  
    253255                }
    254256            };
    255        
     257
    256258        private EditTagDialog(String key, int row, Map<String, Integer> map, final boolean initialFocusOnKey) {
    257259            super(Main.parent, trn("Change value?", "Change values?", map.size()), new String[] {tr("OK"),tr("Cancel")});
    … …  
    262264            this.row = row;
    263265            this.m = map;
    264            
     266
    265267            JPanel mainPanel = new JPanel(new BorderLayout());
    266                    
     268
    267269            String msg = "<html>"+trn("This will change {0} object.",
    268270                    "This will change up to {0} objects.", sel.size(), sel.size())
    … …  
    292294
    293295            final String selection= m.size()!=1?tr("<different>"):m.entrySet().iterator().next().getKey();
    294            
     296
    295297            values = new AutoCompletingComboBox(selection);
    296298            values.setRenderer(cellRenderer);
    … …  
    311313            });
    312314            addFocusAdapter(autocomplete, usedValuesAwareComparator);
    313            
     315
    314316            setContent(mainPanel, false);
    315            
     317
    316318            addWindowListener(new WindowAdapter() {
    317319                @Override
    … …  
    325327            });
    326328        }
    327        
     329
    328330        /**
    329331         * Edit tags of multiple selected objects according to selected ComboBox values
    … …  
    406408        AutoCompletingComboBox values;
    407409        Component componentUnderMouse;
    408        
     410
    409411        public AbstractTagsDialog(Component parent, String title, String[] buttonTexts) {
    410412            super(parent, title, buttonTexts);
    … …  
    422424            // https://bugs.openjdk.java.net/browse/JDK-6200438
    423425            // https://bugs.openjdk.java.net/browse/JDK-6464548
    424            
     426
    425427            setRememberWindowGeometry(getClass().getName() + ".geometry",
    426428                WindowGeometry.centerInWindow(Main.parent, size));
    … …  
    448450            super.setVisible(visible);
    449451        }
    450        
     452
    451453        private void selectACComboBoxSavingUnixBuffer(AutoCompletingComboBox cb) {
    452454            // select compbobox with saving unix system selection (middle mouse paste)
    … …  
    462464            }
    463465        }
    464        
     466
    465467        public void selectKeysComboBox() {
    466468            selectACComboBoxSavingUnixBuffer(keys);
    467469        }
    468        
     470
    469471        public void selectValuesCombobox()   {
    470472            selectACComboBoxSavingUnixBuffer(values);
    471473        }
    472        
     474
    473475        /**
    474476        * Create a focus handling adapter and apply in to the editor component of value
    … …  
    497499           return focus;
    498500        }
    499        
     501
    500502        protected JPopupMenu popupMenu = new JPopupMenu() {
    501503            JCheckBoxMenuItem fixTagLanguageCb = new JCheckBoxMenuItem(
    … …  
    516518    class AddTagsDialog extends AbstractTagsDialog {
    517519        List<JosmAction> recentTagsActions = new ArrayList<>();
    518        
     520
    519521        // Counter of added commands for possible undo
    520522        private int commandCount;
    … …  
    525527            setCancelButton(2);
    526528            configureContextsensitiveHelp("/Dialog/AddValue", true /* show help button */);
    527            
     529
    528530            JPanel mainPanel = new JPanel(new GridBagLayout());
    529531            keys = new AutoCompletingComboBox();
    530532            values = new AutoCompletingComboBox();
    531            
     533
    532534            mainPanel.add(new JLabel("<html>"+trn("This will change up to {0} object.",
    533535                "This will change up to {0} objects.", sel.size(),sel.size())
    … …  
    580582                recentTagsToShow = MAX_LRU_TAGS_NUMBER;
    581583            }
    582            
     584
    583585            // Add tag on Shift-Enter
    584586            mainPanel.getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW).put(
    … …  
    591593                    }
    592594                });
    593                    
     595
    594596            suggestRecentlyAddedTags(mainPanel, recentTagsToShow, focus);
    595            
     597
    596598            setContent(mainPanel, false);
    597            
     599
    598600            selectKeysComboBox();
    599            
     601
    600602            popupMenu.add(new AbstractAction(tr("Set number of recently added tags")) {
    601603                @Override
    … …  
    616618            popupMenu.add(rememberLastTags);
    617619        }
    618        
     620
    619621        private void selectNumberOfTags() {
    620622            String s = JOptionPane.showInputDialog(this, tr("Please enter the number of recently added tags to display"));
    … …  
    630632            JOptionPane.showMessageDialog(this, tr("Please enter integer number between 0 and {0}", MAX_LRU_TAGS_NUMBER));
    631633        }
    632        
     634
    633635        private void suggestRecentlyAddedTags(JPanel mainPanel, int tagsToShow, final FocusAdapter focus) {
    634636            if (!(tagsToShow > 0 && !recentTags.isEmpty()))
    … …  
    636638
    637639            mainPanel.add(new JLabel(tr("Recently added tags")), GBC.eol());
    638            
     640
    639641            int count = 1;
    640642            // We store the maximum number (9) of recent tags to allow dynamic change of number of tags shown in the preferences.
    … …  
    753755            changedKey = key;
    754756        }
    755        
     757
    756758        public void undoAllTagsAdding() {
    757759            Main.main.undoRedo.undo(commandCount);

  • trunk/src/org/openstreetmap/josm/gui/download/BookmarkSelection.java

    r7021 r7029  
    221221                return;
    222222            for (Object sel: sels) {
    223                 ((DefaultListModel)bookmarks.getModel()).removeElement(sel);
     223                ((DefaultListModel<Bookmark>)bookmarks.getModel()).removeElement(sel);
    224224            }
    225225            bookmarks.save();

  • trunk/src/org/openstreetmap/josm/gui/io/SaveLayersDialog.java

    r7017 r7029  
    3737import javax.swing.JScrollPane;
    3838import javax.swing.KeyStroke;
     39import javax.swing.ListCellRenderer;
    3940import javax.swing.WindowConstants;
    4041import javax.swing.event.TableModelEvent;
    … …  
    175176            lstLayers = new JList<>();
    176177            lstLayers.setCellRenderer(
    177                     new DefaultListCellRenderer() {
     178                    new ListCellRenderer<SaveLayerInfo>() {
     179                        final DefaultListCellRenderer def = new DefaultListCellRenderer();
    178180                        @Override
    179                         public Component getListCellRendererComponent(JList list, Object value, int index,
     181                        public Component getListCellRendererComponent(JList<? extends SaveLayerInfo> list, SaveLayerInfo info, int index,
    180182                                boolean isSelected, boolean cellHasFocus) {
    181                             SaveLayerInfo info = (SaveLayerInfo)value;
    182                             setIcon(info.getLayer().getIcon());
    183                             setText(info.getName());
    184                             return this;
     183                            def.setIcon(info.getLayer().getIcon());
     184                            def.setText(info.getName());
     185                            return def;
    185186                        }
    186187                    }

  • trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java

    r7005 r7029  
    115115                String defaultValue = layerName == null ? getDefaultLabelPattern():"";
    116116                TemplateEntryProperty parent = layerName == null ? null : forMarker(null);
    117                 try {
    118                     result = new TemplateEntryProperty(key, defaultValue, parent);
    119                     CACHE.put(key, result);
    120                 } catch (ParseError e) {
    121                     Main.warn("Unable to parse template engine pattern ''{0}'' for property {1}", defaultValue, key);
    122                 }
     117                result = new TemplateEntryProperty(key, defaultValue, parent);
     118                CACHE.put(key, result);
    123119            }
    124120            return result;
    … …  
    134130                String defaultValue = layerName == null?"?{ '{name}' | '{desc}' | '{" + Marker.MARKER_FORMATTED_OFFSET + "}' }":"";
    135131                TemplateEntryProperty parent = layerName == null ? null : forAudioMarker(null);
    136                 try {
    137                     result = new TemplateEntryProperty(key, defaultValue, parent);
    138                     CACHE.put(key, result);
    139                 } catch (ParseError e) {
    140                     Main.warn("Unable to parse template engine pattern ''{0}'' for property {1}", defaultValue, key);
    141                 }
     132                result = new TemplateEntryProperty(key, defaultValue, parent);
     133                CACHE.put(key, result);
    142134            }
    143135            return result;
    … …  
    146138        private TemplateEntryProperty parent;
    147139
    148 
    149         private TemplateEntryProperty(String key, String defaultValue, TemplateEntryProperty parent) throws ParseError {
     140        private TemplateEntryProperty(String key, String defaultValue, TemplateEntryProperty parent) {
    150141            super(key, defaultValue);
    151142            this.parent = parent;

  • trunk/src/org/openstreetmap/josm/gui/mappaint/Cascade.java

    r7005 r7029  
    181181    public Cascade clone() {
    182182        @SuppressWarnings("unchecked")
    183         HashMap<String, Object> clonedProp = (HashMap) ((HashMap) this.prop).clone();
     183        HashMap<String, Object> clonedProp = (HashMap<String, Object>) ((HashMap) this.prop).clone();
    184184        Cascade c = new Cascade();
    185185        c.prop = clonedProp;

  • trunk/src/org/openstreetmap/josm/io/OsmExporter.java

    r7005 r7029  
    4343    }
    4444
    45     public void exportData(File file, Layer layer, boolean noBackup) throws IOException {
     45    public void exportData(File file, Layer layer, boolean noBackup) throws IllegalArgumentException {
    4646        if (layer instanceof OsmDataLayer) {
    4747            save(file, (OsmDataLayer) layer, noBackup);

  • trunk/src/org/openstreetmap/josm/io/session/GpxTracksSessionExporter.java

    r6552 r7029  
    179179    }
    180180
    181     protected void addDataFile(OutputStream out) throws IOException {
     181    protected void addDataFile(OutputStream out) {
    182182        Writer writer = new OutputStreamWriter(out, Utils.UTF_8);
    183183        GpxWriter w = new GpxWriter(new PrintWriter(writer));

  • trunk/src/org/openstreetmap/josm/io/session/MarkerSessionExporter.java

    r6552 r7029  
    1717import javax.swing.SwingConstants;
    1818
    19 import org.openstreetmap.josm.tools.Utils;
    20 import org.w3c.dom.Element;
    21 
    2219import org.openstreetmap.josm.Main;
    2320import org.openstreetmap.josm.data.gpx.GpxData;
    … …  
    2825import org.openstreetmap.josm.io.session.SessionWriter.ExportSupport;
    2926import org.openstreetmap.josm.tools.GBC;
     27import org.openstreetmap.josm.tools.Utils;
     28import org.w3c.dom.Element;
    3029
    3130public class MarkerSessionExporter implements SessionLayerExporter {
    … …  
    8584    }
    8685
    87     protected void addDataFile(OutputStream out) throws IOException {
     86    protected void addDataFile(OutputStream out) {
    8887        Writer writer = new OutputStreamWriter(out, Utils.UTF_8);
    8988        MarkerWriter w = new MarkerWriter(new PrintWriter(writer));

  • trunk/src/org/openstreetmap/josm/io/session/OsmDataSessionExporter.java

    r6890 r7029  
    3333import javax.swing.SwingConstants;
    3434
    35 import org.openstreetmap.josm.tools.Utils;
    36 import org.w3c.dom.Element;
    37 
    3835import org.openstreetmap.josm.actions.SaveAction;
    3936import org.openstreetmap.josm.gui.layer.Layer;
    … …  
    4643import org.openstreetmap.josm.tools.GBC;
    4744import org.openstreetmap.josm.tools.ImageProvider;
     45import org.openstreetmap.josm.tools.Utils;
     46import org.w3c.dom.Element;
    4847
    4948public class OsmDataSessionExporter implements SessionLayerExporter {
    … …  
    213212    }
    214213
    215     protected void addDataFile(OutputStream out) throws IOException {
     214    protected void addDataFile(OutputStream out) {
    216215        Writer writer = new OutputStreamWriter(out, Utils.UTF_8);
    217216        OsmWriter w = OsmWriterFactory.createOsmWriter(new PrintWriter(writer), false, layer.data.getVersion());

  • trunk/src/org/openstreetmap/josm/tools/AudioUtil.java

    r7025 r7029  
    44import java.io.File;
    55import java.io.IOException;
    6 import java.net.MalformedURLException;
    76import java.net.URL;
    87

  • trunk/src/org/openstreetmap/josm/tools/OpenBrowser.java

    r7026 r7029  
    66import java.awt.Desktop;
    77import java.io.IOException;
    8 import java.net.MalformedURLException;
    98import java.net.URI;
    109
    … …  
    2322        // Hide default constructor for utils classes
    2423    }
    25    
     24
    2625    private static void displayUrlFallback(URI uri) throws IOException {
    2726        if (Main.platform == null)
    … …  
    3130
    3231    /**
    33      * Displays an external URI using platform associated software. 
    34      * A web resource will launch platform's browser, an audio file URI will launch audio player, etc. 
     32     * Displays an external URI using platform associated software.
     33     * A web resource will launch platform's browser, an audio file URI will launch audio player, etc.
    3534     * @param uri The URI to display
    3635     * @return <code>null</code> for success or a string in case of an error.
    … …  
    7877
    7978    /**
    80      * Displays an external URL using platform associated software. 
    81      * A web resource will launch platform's browser, an audio file URL will launch audio player, etc. 
     79     * Displays an external URL using platform associated software.
     80     * A web resource will launch platform's browser, an audio file URL will launch audio player, etc.
    8281     * @param url The URL to display
    8382     * @return <code>null</code> for success or a string in case of an error.
Note: See TracChangeset for help on using the changeset viewer.