Changeset 7021 in josm for trunk


Ignore:
Timestamp:
2014-04-29T01:18:18+02:00 (6 years ago)
Author:
Don-vip
Message:

see #8465 - enable -Xlint:cast and fix associated warnings

Location:
trunk
Files:
12 edited

Legend:

Unmodified
Added
Removed
  • trunk/build.xml

    r7019 r7021  
    206206        <javac srcdir="src" excludes="com/**,oauth/**,org/apache/commons/codec/**,org/openstreetmap/gui/jmapviewer/Demo.java"
    207207                destdir="build" target="1.7" source="1.7" debug="on" includeantruntime="false" encoding="UTF-8">
     208            <compilerarg value="-Xlint:cast"/>
    208209            <compilerarg value="-Xlint:deprecation"/>
    209210            <compilerarg value="-Xlint:unchecked"/>
  • trunk/src/org/openstreetmap/josm/gui/OsmPrimitivRenderer.java

    r7001 r7021  
    4545    public Component getListCellRendererComponent(JList<? extends OsmPrimitive> list, OsmPrimitive value, int index, boolean isSelected, boolean cellHasFocus) {
    4646        Component def = defaultListCellRenderer.getListCellRendererComponent(list, value, index, isSelected, cellHasFocus);
    47         return renderer(def, (OsmPrimitive) value);
     47        return renderer(def, value);
    4848    }
    4949
  • trunk/src/org/openstreetmap/josm/gui/dialogs/RelationListDialog.java

    r7005 r7021  
    316316
    317317        protected void setCurrentRelationAsSelection() {
    318             Main.main.getCurrentDataSet().setSelected((Relation)displaylist.getSelectedValue());
     318            Main.main.getCurrentDataSet().setSelected(displaylist.getSelectedValue());
    319319        }
    320320
  • trunk/src/org/openstreetmap/josm/gui/dialogs/SelectionListDialog.java

    r7010 r7021  
    203203                OsmDataLayer layer = Main.main.getEditLayer();
    204204                if (layer == null) return;
    205                 layer.data.setSelected(Collections.singleton((OsmPrimitive)model.getElementAt(idx)));
     205                layer.data.setSelected(Collections.singleton(model.getElementAt(idx)));
    206206            } else if (highlightEnabled && Main.isDisplayingMapView()) {
    207                 if (helper.highlightOnly((OsmPrimitive)model.getElementAt(idx))) {
     207                if (helper.highlightOnly(model.getElementAt(idx))) {
    208208                    Main.map.mapView.repaint();
    209209                }
  • trunk/src/org/openstreetmap/josm/gui/download/BookmarkSelection.java

    r7020 r7021  
    114114            @Override
    115115            public void valueChanged(ListSelectionEvent e) {
    116                 Bookmark b = (Bookmark)bookmarks.getSelectedValue();
     116                Bookmark b = bookmarks.getSelectedValue();
    117117                if (b != null) {
    118118                    gui.boundingBoxChanged(b.getArea(),BookmarkSelection.this);
     
    282282            if (idx < 0 || idx >= bookmarks.getModel().getSize())
    283283                return;
    284             Bookmark b = (Bookmark)bookmarks.getModel().getElementAt(idx);
     284            Bookmark b = bookmarks.getModel().getElementAt(idx);
    285285            parent.startDownload(b.getArea());
    286286        }
  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

    r7017 r7021  
    11751175        // Time between the track point and the previous one, 5 sec if first point, i.e. photos take
    11761176        // 5 sec before the first track point can be assumed to be take at the starting position
    1177         long interval = prevWpTime > 0 ? ((long)Math.abs(curWpTime - prevWpTime)) : 5*1000;
     1177        long interval = prevWpTime > 0 ? (Math.abs(curWpTime - prevWpTime)) : 5*1000;
    11781178        int ret = 0;
    11791179
  • trunk/src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java

    r7020 r7021  
    623623                @Override
    624624                public Component getListCellRendererComponent(JList<? extends ActionDefinition> list,
    625                         ActionDefinition value, int index, boolean isSelected, boolean cellHasFocus) {
     625                        ActionDefinition action, int index, boolean isSelected, boolean cellHasFocus) {
    626626                    String s;
    627627                    Icon i;
    628                     ActionDefinition action = (ActionDefinition)value;
    629628                    if (!action.isSeparator()) {
    630629                        s = action.getDisplayName();
     
    646645                    if (sel) {
    647646                        actionsTree.clearSelection();
    648                         ActionDefinition action = (ActionDefinition) selected.get(selectedList.getSelectedIndex());
     647                        ActionDefinition action = selected.get(selectedList.getSelectedIndex());
    649648                        actionParametersModel.setCurrentAction(action);
    650649                        actionParametersPanel.setVisible(actionParametersModel.getRowCount() > 0);
     
    864863            ActionParser parser = new ActionParser(null);
    865864            for (int i = 0; i < selected.size(); ++i) {
    866                 ActionDefinition action = (ActionDefinition)selected.get(i);
     865                ActionDefinition action = selected.get(i);
    867866                if (action.isSeparator()) {
    868867                    t.add("|");
  • trunk/src/org/openstreetmap/josm/gui/preferences/display/LafPreference.java

    r7020 r7021  
    7878        String laf = Main.pref.get("laf", Main.platform.getDefaultStyle());
    7979        for (int i = 0; i < lafCombo.getItemCount(); ++i) {
    80             if (((LookAndFeelInfo)lafCombo.getItemAt(i)).getClassName().equals(laf)) {
     80            if (lafCombo.getItemAt(i).getClassName().equals(laf)) {
    8181                lafCombo.setSelectedIndex(i);
    8282                break;
  • trunk/src/org/openstreetmap/josm/gui/preferences/plugin/PluginPreference.java

    r7005 r7021  
    567567            List<String> ret = new ArrayList<>(model.getSize());
    568568            for (int i=0; i< model.getSize();i++){
    569                 ret.add((String)model.get(i));
     569                ret.add(model.get(i));
    570570            }
    571571            return ret;
  • trunk/src/org/openstreetmap/josm/gui/preferences/projection/ProjectionPreference.java

    r7015 r7021  
    300300
    301301        for (int i = 0; i < coordinatesCombo.getItemCount(); ++i) {
    302             if (((CoordinateFormat)coordinatesCombo.getItemAt(i)).name().equals(PROP_COORDINATES.get())) {
     302            if (coordinatesCombo.getItemAt(i).name().equals(PROP_COORDINATES.get())) {
    303303                coordinatesCombo.setSelectedIndex(i);
    304304                break;
     
    445445        ProjectionChoice pc = null;
    446446        for (int i = 0; i < projectionCombo.getItemCount(); ++i) {
    447             ProjectionChoice pc1 = (ProjectionChoice) projectionCombo.getItemAt(i);
     447            ProjectionChoice pc1 = projectionCombo.getItemAt(i);
    448448            pc1.setPreferences(getSubprojectionPreference(pc1));
    449449            if (pc1.getId().equals(PROP_PROJECTION.get())) {
  • trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletingComboBox.java

    r7015 r7021  
    136136            AutoCompletionListItem bestItem = null;
    137137            for (int i = 0, n = model.getSize(); i < n; i++) {
    138                 AutoCompletionListItem currentItem = (AutoCompletionListItem) model.getElementAt(i);
     138                AutoCompletionListItem currentItem = model.getElementAt(i);
    139139                if (currentItem.getValue().equals(pattern))
    140140                    return currentItem;
     
    224224            // find the string in the model or create a new item
    225225            for (int i=0; i< getModel().getSize(); i++) {
    226                 AutoCompletionListItem acItem = (AutoCompletionListItem) getModel().getElementAt(i);
     226                AutoCompletionListItem acItem = getModel().getElementAt(i);
    227227                if (s.equals(acItem.getValue())) {
    228228                    super.setSelectedItem(acItem);
  • trunk/src/org/openstreetmap/josm/gui/widgets/ComboBoxHistory.java

    r7015 r7021  
    2929    @Override
    3030    public void addElement(AutoCompletionListItem o) {
    31         String newEntry = ((AutoCompletionListItem)o).getValue();
     31        String newEntry = o.getValue();
    3232
    3333        // if history contains this object already, delete it,
    3434        // so that it looks like a move to the top
    3535        for (int i = 0; i < getSize(); i++) {
    36             String oldEntry = ((AutoCompletionListItem) getElementAt(i)).getValue();
     36            String oldEntry = getElementAt(i).getValue();
    3737            if(oldEntry.equals(newEntry)) {
    3838                removeElementAt(i);
     
    7575            public AutoCompletionListItem next() {
    7676                position++;
    77                 return (AutoCompletionListItem)getElementAt(position);
     77                return getElementAt(position);
    7878            }
    7979        };
Note: See TracChangeset for help on using the changeset viewer.