Ignore:
Timestamp:
2016-06-01T23:17:40+02:00 (3 years ago)
Author:
Don-vip
Message:

sonar - squid:S1854 - Dead stores should be removed

Location:
trunk/src/org/openstreetmap/josm/gui/history
Files:
6 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java

    r10212 r10308  
    763763        l.data.addDataSetListener(this);
    764764        OsmPrimitive primitive = history != null ? l.data.getPrimitiveById(history.getId(), history.getType()) : null;
    765         HistoryOsmPrimitive latest;
     765        HistoryOsmPrimitive newLatest;
    766766        if (canShowAsLatest(primitive)) {
    767             latest = new HistoryPrimitiveBuilder().build(primitive);
     767            newLatest = new HistoryPrimitiveBuilder().build(primitive);
    768768        } else {
    769             latest = null;
    770         }
    771         setLatest(latest);
     769            newLatest = null;
     770        }
     771        setLatest(newLatest);
    772772        fireModelChange();
    773773    }
  • trunk/src/org/openstreetmap/josm/gui/history/NodeListTableCellRenderer.java

    r10021 r10308  
    3333    protected void renderNode(TwoColumnDiff.Item item, boolean isSelected) {
    3434        String text = "";
    35         Color bgColor = Color.WHITE;
    3635        setIcon(nodeIcon);
    3736        if (item.value != null) {
    3837            text = tr("Node {0}", item.value.toString());
    3938        }
    40         bgColor = item.state.getColor();
     39        Color bgColor = item.state.getColor();
    4140        if (item.state == DiffItemType.EMPTY) {
    4241            text = "";
  • trunk/src/org/openstreetmap/josm/gui/history/NodeListTableColumnModel.java

    r5835 r10308  
    77import javax.swing.table.TableColumn;
    88
    9 
    109/**
    1110 * The {@link javax.swing.table.TableColumnModel} for the table with the list of nodes.
    12  *
    13  *
     11 * @since 1709
    1412 */
    1513public class NodeListTableColumnModel extends DefaultTableColumnModel {
     14
     15    /**
     16     * Constructs a new {@code NodeListTableColumnModel}.
     17     */
     18    public NodeListTableColumnModel() {
     19        createColumns();
     20    }
     21
    1622    protected void createColumns() {
    17         TableColumn col = null;
    1823        NodeListTableCellRenderer renderer = new NodeListTableCellRenderer();
    1924
    2025        // column 0 - Version
    21         col = new TableColumn(0);
     26        TableColumn col = new TableColumn(0);
    2227        col.setHeaderValue(tr("Nodes"));
    2328        col.setCellRenderer(renderer);
    2429        addColumn(col);
    2530    }
    26 
    27     public NodeListTableColumnModel() {
    28         createColumns();
    29     }
    3031}
  • trunk/src/org/openstreetmap/josm/gui/history/RelationMemberTableColumnModel.java

    r5835 r10308  
    99/**
    1010 * The {@link javax.swing.table.TableColumnModel} for the table with the list of relation members.
    11  *
     11 * @since 1709
    1212 */
    1313public class RelationMemberTableColumnModel extends DefaultTableColumnModel {
     14
     15    /**
     16     * Constructs a new {@code RelationMemberTableColumnModel}.
     17     */
     18    public RelationMemberTableColumnModel() {
     19        createColumns();
     20    }
     21
    1422    protected void createColumns() {
    15         TableColumn col = null;
    1623        RelationMemberListTableCellRenderer renderer = new RelationMemberListTableCellRenderer();
    1724
    1825        // column 0 - Version
    19         col = new TableColumn(0);
     26        TableColumn col = new TableColumn(0);
    2027        col.setHeaderValue(tr("Role"));
    2128        col.setCellRenderer(renderer);
     
    2835        addColumn(col);
    2936    }
    30 
    31     public RelationMemberTableColumnModel() {
    32         createColumns();
    33     }
    3437}
  • trunk/src/org/openstreetmap/josm/gui/history/TagTableColumnModel.java

    r7822 r10308  
    1212 */
    1313public class TagTableColumnModel extends DefaultTableColumnModel {
     14
     15    /**
     16     * Constructs a new {@code TagTableColumnModel}.
     17     */
     18    public TagTableColumnModel() {
     19        createColumns();
     20    }
     21
    1422    protected void createColumns() {
    15         TableColumn col = null;
    16 
    1723        TagTableCellRenderer renderer = new TagTableCellRenderer();
    1824
    1925        // column 0 - Key
    20         col = new TableColumn(0);
     26        TableColumn col = new TableColumn(0);
    2127        col.setHeaderValue(tr("Key"));
    2228        col.setCellRenderer(renderer);
     
    2834        col.setCellRenderer(renderer);
    2935        addColumn(col);
    30 
    31     }
    32 
    33     /**
    34      * Constructs a new {@code TagTableColumnModel}.
    35      */
    36     public TagTableColumnModel() {
    37         createColumns();
    3836    }
    3937}
  • trunk/src/org/openstreetmap/josm/gui/history/VersionTableColumnModel.java

    r9983 r10308  
    1111/**
    1212 * The {@link TableColumnModel} for the table with the list of versions
    13  *
     13 * @since 1709
    1414 */
    1515public class VersionTableColumnModel extends DefaultTableColumnModel {
     
    2626    public static final int COL_USER = 4;
    2727
     28    /**
     29     * Creates a new {@code VersionTableColumnModel}.
     30     */
     31    public VersionTableColumnModel() {
     32        createColumns();
     33    }
     34
    2835    protected void createColumns() {
    29         TableColumn col = null;
    3036        VersionTable.RadioButtonRenderer bRenderer = new VersionTable.RadioButtonRenderer();
    3137
    3238        // column 0 - Version
    33         col = new TableColumn(COL_VERSION);
     39        TableColumn col = new TableColumn(COL_VERSION);
    3440        /* translation note: 3 letter abbr. for "Version" */
    3541        col.setHeaderValue(tr("Ver"));
     
    6268        addColumn(col);
    6369    }
    64 
    65     /**
    66      * Creates a new {@code VersionTableColumnModel}.
    67      */
    68     public VersionTableColumnModel() {
    69         createColumns();
    70     }
    7170}
Note: See TracChangeset for help on using the changeset viewer.