Changeset 10308 in josm


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
Files:
57 edited

Legend:

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

    r8870 r10308  
    247247        int nodeCount = nodes.size();
    248248        // Search first fixed node
    249         int startPosition = 0;
     249        int startPosition;
    250250        for (startPosition = 0; startPosition < nodeCount; startPosition++) {
    251251            if (fixNodes.contains(nodes.get(startPosition % nodeCount)))
  • trunk/src/org/openstreetmap/josm/actions/AlignInLineAction.java

    r9087 r10308  
    173173
    174174        try {
    175             Command cmd = null;
     175            Command cmd;
    176176            // Decide what to align based on selection:
    177177
     
    182182                // Only 1 node selected -> align this node relative to referers way
    183183                Node selectedNode = selectedNodes.get(0);
    184                 List<Way> involvedWays = null;
     184                List<Way> involvedWays;
    185185                if (selectedWays.isEmpty())
    186186                    // No selected way, all way containing this node are used
  • trunk/src/org/openstreetmap/josm/actions/JoinAreasAction.java

    r10174 r10308  
    775775     * @param isInner - if true, reverts the direction (for multipolygon islands)
    776776     * @return list of parts, marked with the inside orientation.
    777      * @throws IllegalArgumentException if parts is empty
     777     * @throws IllegalArgumentException if parts is empty or not circular
    778778     */
    779779    private static List<WayInPolygon> markWayInsideSide(List<Way> parts, boolean isInner) {
     
    785785
    786786            if (!parts.get(pos).lastNode().equals(parts.get((pos + 1) % parts.size()).firstNode()))
    787                 throw new RuntimeException("Way not circular");
     787                throw new IllegalArgumentException("Way not circular");
    788788
    789789            nextWayMap.put(parts.get(pos), parts.get((pos + 1) % parts.size()));
     
    818818
    819819        if (topNode.equals(topWay.firstNode()) || topNode.equals(topWay.lastNode())) {
    820             Node headNode = null; // the node at junction
    821             Node prevNode = null; // last node from previous path
    822             wayClockwise = false;
     820            Node headNode; // the node at junction
     821            Node prevNode; // last node from previous path
    823822
    824823            //node is in split point - find the outermost way from this point
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmChangeTask.java

    r10001 r10308  
    149149                    HistoryOsmPrimitive hp = history.getByDate(date);
    150150                    if (hp != null) {
    151                         PrimitiveData data = null;
     151                        PrimitiveData data;
    152152
    153153                        switch (p.getType()) {
  • trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

    r10303 r10308  
    398398
    399399        boolean newNode = false;
    400         Node n = null;
    401 
    402         n = Main.map.mapView.getNearestNode(mousePos, OsmPrimitive.isSelectablePredicate);
     400        Node n = Main.map.mapView.getNearestNode(mousePos, OsmPrimitive.isSelectablePredicate);
    403401        if (ctrl) {
    404402            Iterator<Way> it = getCurrentDataSet().getSelectedWays().iterator();
  • trunk/src/org/openstreetmap/josm/actions/mapmode/ExtrudeAction.java

    r10115 r10308  
    684684            wnew.addNode(insertionPoint, n2New);
    685685            wayWasModified = true;
    686             insertionPoint++;
    687686            cmds.add(new AddCommand(n2New));
    688687            changedNodes.add(n2New);
  • trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java

    r10179 r10308  
    11961196                Pair<Node, Node> vnp = null, wnp = new Pair<>(null, null);
    11971197
    1198                 Way w = null;
    11991198                for (WaySegment ws : mv.getNearestWaySegments(p, mv.isSelectablePredicate)) {
    1200                     w = ws.way;
     1199                    Way w = ws.way;
    12011200
    12021201                    wnp.a = w.getNode(ws.lowerIndex);
  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r10300 r10308  
    680680                }
    681681            } else {
    682                 String mv = null;
     682                String mv;
    683683
    684684                if ("timestamp".equals(key) && osm instanceof OsmPrimitive) {
  • trunk/src/org/openstreetmap/josm/command/ChangePropertyCommand.java

    r10216 r10308  
    154154        if (objects.size() == 1 && tags.size() == 1) {
    155155            OsmPrimitive primitive = objects.get(0);
    156             String msg = "";
     156            String msg;
    157157            Map.Entry<String, String> entry = tags.entrySet().iterator().next();
    158158            if (entry.getValue() == null) {
  • trunk/src/org/openstreetmap/josm/corrector/TagCorrector.java

    r10125 r10308  
    146146
    147147                    // create the clone
    148                     OsmPrimitive clone = null;
     148                    OsmPrimitive clone;
    149149                    if (primitive instanceof Way) {
    150150                        clone = new Way((Way) primitive);
  • trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java

    r10212 r10308  
    256256
    257257    private void finishLoading(LoadResult result) {
    258         Set<ICachedLoaderListener> listeners = null;
     258        Set<ICachedLoaderListener> listeners;
    259259        synchronized (inProgress) {
    260260            listeners = inProgress.remove(getUrlNoException().toString());
  • trunk/src/org/openstreetmap/josm/data/osm/DataSetMerger.java

    r9961 r10308  
    111111        // the target dataset. Create a clone and add it to the target dataset.
    112112        //
    113         OsmPrimitive target = null;
     113        OsmPrimitive target;
    114114        switch(source.getType()) {
    115115        case NODE: target = source.isNew() ? new Node() : new Node(source.getId()); break;
  • trunk/src/org/openstreetmap/josm/data/osm/FilterWorker.java

    r9665 r10308  
    2424     * Apply the filters to the primitives of the data set.
    2525     *
    26      * @param all the collection of primitives for that the filter state should
    27      * be updated
     26     * @param all the collection of primitives for that the filter state should be updated
    2827     * @param filterMatcher the FilterMatcher
    2928     * @return true, if the filter state (normal / disabled / hidden)
     
    3130     */
    3231    public static boolean executeFilters(Collection<OsmPrimitive> all, FilterMatcher filterMatcher) {
    33         boolean changed = false;
     32        boolean changed;
    3433        // first relations, then ways and nodes last; this is required to resolve dependencies
    3534        changed = doExecuteFilters(Utils.filter(all, OsmPrimitive.relationPredicate), filterMatcher);
  • trunk/src/org/openstreetmap/josm/data/osm/event/DatasetEventManager.java

    r10043 r10308  
    3636                DataSet dataSet = null;
    3737                AbstractDatasetChangedEvent consolidatedEvent = null;
    38                 AbstractDatasetChangedEvent event = null;
     38                AbstractDatasetChangedEvent event;
    3939
    4040                while ((event = eventsInEDT.poll()) != null) {
  • trunk/src/org/openstreetmap/josm/data/projection/datum/NTV2Util.java

    r9981 r10308  
    7272     */
    7373    public static float getFloat(byte[] b, boolean bigEndian) {
    74         int i = 0;
     74        int i;
    7575        if (bigEndian) {
    7676            i = getIntBE(b, 0);
     
    8989     */
    9090    public static double getDouble(byte[] b, boolean bigEndian) {
    91         int i = 0;
    92         int j = 0;
     91        int i;
     92        int j;
    9393        if (bigEndian) {
    9494            i = getIntBE(b, 0);
     
    9898            j = getIntLE(b, 0);
    9999        }
    100         long l = ((long) i << 32) |
    101         (j & 0x00000000FFFFFFFFL);
     100        long l = ((long) i << 32) | (j & 0x00000000FFFFFFFFL);
    102101        return Double.longBitsToDouble(l);
    103102    }
  • trunk/src/org/openstreetmap/josm/data/projection/proj/SwissObliqueMercator.java

    r10001 r10308  
    102102        double lambda = l / alpha;
    103103        double phi = b;
    104         double s = 0;
    105104
    106105        double prevPhi = -1000;
     
    111110                throw new RuntimeException("Two many iterations");
    112111            prevPhi = phi;
    113             s = 1 / alpha * (log(tan(PI / 4 + b / 2)) - k) + ellps.e
     112            double s = 1 / alpha * (log(tan(PI / 4 + b / 2)) - k) + ellps.e
    114113            * log(tan(PI / 4 + asin(ellps.e * sin(phi)) / 2));
    115114            phi = 2 * atan(exp(s)) - PI / 2;
  • trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java

    r10043 r10308  
    150150                for (Role role: r.roles) {
    151151                    String key = role.key;
    152                     List<Role> roleGroup = null;
     152                    List<Role> roleGroup;
    153153                    if (allroles.containsKey(key)) {
    154154                        roleGroup = allroles.get(key).roles;
  • trunk/src/org/openstreetmap/josm/gui/DefaultNameFormatter.java

    r10242 r10308  
    210210        StringBuilder name = new StringBuilder();
    211211
    212         char mark = 0;
     212        char mark;
    213213        // If current language is left-to-right (almost all languages)
    214214        if (ComponentOrientation.getOrientation(Locale.getDefault()).isLeftToRight()) {
  • trunk/src/org/openstreetmap/josm/gui/ExtendedDialog.java

    r10035 r10308  
    386386        }
    387387
    388         gc.gridy = y++;
     388        gc.gridy = y;
    389389        gc.anchor = GridBagConstraints.CENTER;
    390390            gc.insets = new Insets(5, 5, 5, 5);
  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/PairTable.java

    r8509 r10308  
    1616    private final transient ListMergeModel<? extends PrimitiveId> model;
    1717
     18    /**
     19     * Constructs a new {@code PairTable}.
     20     * @param name table name
     21     * @param model merge model
     22     * @param dm table model
     23     * @param cm column model
     24     * @param sm selection model
     25     */
    1826    public PairTable(String name, ListMergeModel<? extends PrimitiveId> model,
    1927            OsmPrimitivesTableModel dm, TableColumnModel cm, ListSelectionModel sm) {
  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListColumnModel.java

    r6890 r10308  
    88import javax.swing.table.TableColumn;
    99
     10/**
     11 * Column model used in {@link NodeListTable}.
     12 * @since 1622
     13 */
    1014public class NodeListColumnModel extends DefaultTableColumnModel {
     15
     16    /**
     17     * Constructs a new {@code NodeListColumnModel}.
     18     * @param renderer table cell renderer
     19     */
     20    public NodeListColumnModel(TableCellRenderer renderer) {
     21        createColumns(renderer);
     22    }
    1123
    1224    protected final void createColumns(TableCellRenderer renderer) {
    1325
    14         TableColumn col = null;
    15 
    1626        // column 0 - Row num
    17         col = new TableColumn(0);
     27        TableColumn col = new TableColumn(0);
    1828        col.setHeaderValue("");
    1929        col.setResizable(true);
     
    3040        addColumn(col);
    3141    }
    32 
    33     public NodeListColumnModel(TableCellRenderer renderer) {
    34         createColumns(renderer);
    35     }
    3642}
  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListTable.java

    r7937 r10308  
    1212public class NodeListTable extends PairTable {
    1313
     14    /**
     15     * Constructs a new {@code NodeListTable}.
     16     * @param name table name
     17     * @param model node merge model
     18     * @param dm table model
     19     * @param sm selection model
     20     */
    1421    public NodeListTable(String name, ListMergeModel<Node> model, OsmPrimitivesTableModel dm, ListSelectionModel sm) {
    1522        super(name, model, dm, new NodeListColumnModel(new NodeListTableCellRenderer()), sm);
  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListTableCellRenderer.java

    r9243 r10308  
    2020/**
    2121 * This is the {@link TableCellRenderer} used in the node tables of {@link NodeListMerger}.
    22  *
     22 * @since 1622
    2323 */
    24 public  class NodeListTableCellRenderer extends JLabel implements TableCellRenderer {
     24public class NodeListTableCellRenderer extends JLabel implements TableCellRenderer {
    2525
    2626    private final ImageIcon icon;
     
    118118            default:
    119119                // should not happen
    120                 throw new RuntimeException(MessageFormat.format("Unexpected column index. Got {0}.", column));
     120                throw new IllegalArgumentException(MessageFormat.format("Unexpected column index. Got {0}.", column));
    121121            }
    122122        }
  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/relation/RelationMemberListColumnModel.java

    r9059 r10308  
    77import javax.swing.table.TableColumn;
    88
     9/**
     10 * Column model used in {@link RelationMemberTable}.
     11 * @since 1622
     12 */
    913public class RelationMemberListColumnModel extends DefaultTableColumnModel {
    1014
     15    /**
     16     * Constructs a new {@code RelationMemberListColumnModel}.
     17     */
     18    public RelationMemberListColumnModel() {
     19        createColumns();
     20    }
     21
    1122    protected final void createColumns() {
    12         TableColumn col = null;
    1323        RelationMemberTableCellRenderer renderer = new RelationMemberTableCellRenderer();
    1424
    1525        // column 0 - Row num
    16         col = new TableColumn(0);
     26        TableColumn col = new TableColumn(0);
    1727        col.setHeaderValue("");
    1828        col.setResizable(true);
     
    3848        addColumn(col);
    3949    }
    40 
    41     public RelationMemberListColumnModel() {
    42         createColumns();
    43     }
    4450}
  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/relation/RelationMemberTable.java

    r7937 r10308  
    1212public class RelationMemberTable extends PairTable {
    1313
     14    /**
     15     * Constructs a new {@code RelationMemberTable}.
     16     * @param name table name
     17     * @param model relation member merge model
     18     * @param dm table model
     19     * @param sm selection model
     20     */
    1421    public RelationMemberTable(String name, ListMergeModel<RelationMember> model, OsmPrimitivesTableModel dm, ListSelectionModel sm) {
    1522        super(name, model, dm, new RelationMemberListColumnModel(), sm);
  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/tags/TagMerger.java

    r9847 r10308  
    360360            if (e.getClickCount() != 2)
    361361                return;
    362             JTable table = null;
     362            JTable table;
    363363            MergeDecisionType mergeDecision;
    364364
  • trunk/src/org/openstreetmap/josm/gui/conflict/tags/RelationMemberConflictResolverColumnModel.java

    r8308 r10308  
    6666        RelationMemberConflictDecisionEditor decisionEditor = new RelationMemberConflictDecisionEditor();
    6767
    68         TableColumn col = null;
    69 
    7068        // column 0 - Relation
    71         col = new TableColumn(0);
     69        TableColumn col = new TableColumn(0);
    7270        col.setHeaderValue("Relation");
    7371        col.setResizable(true);
  • trunk/src/org/openstreetmap/josm/gui/dialogs/DeleteFromRelationConfirmationDialog.java

    r10300 r10308  
    275275
    276276        protected final void createColumns() {
    277             TableColumn col = null;
    278277
    279278            // column 0 - To Delete
    280             col = new TableColumn(0);
     279            TableColumn col = new TableColumn(0);
    281280            col.setHeaderValue(tr("To delete"));
    282281            col.setResizable(true);
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetContentTableColumnModel.java

    r7715 r10308  
    1111/**
    1212 * The column model for the changeset content
    13  *
     13 * @since 2689
    1414 */
    1515public class ChangesetContentTableColumnModel extends DefaultTableColumnModel {
    1616
     17    /**
     18     * Constructs a new {@code ChangesetContentTableColumnModel}.
     19     */
     20    public ChangesetContentTableColumnModel() {
     21        createColumns();
     22    }
     23
    1724    protected void createColumns() {
    18         TableColumn col = null;
    1925        ChangesetContentTableCellRenderer renderer = new ChangesetContentTableCellRenderer();
    2026        // column 0 - type
    21         col = new TableColumn(0);
     27        TableColumn col = new TableColumn(0);
    2228        col.setHeaderValue("");
    2329        col.setResizable(true);
     
    4551        addColumn(col);
    4652    }
    47 
    48     /**
    49      * Constructs a new {@code ChangesetContentTableColumnModel}.
    50      */
    51     public ChangesetContentTableColumnModel() {
    52         createColumns();
    53     }
    5453}
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetDiscussionTableColumnModel.java

    r7937 r10308  
    1313public class ChangesetDiscussionTableColumnModel extends DefaultTableColumnModel {
    1414
     15    /**
     16     * Constructs a new {@code ChangesetContentTableColumnModel}.
     17     */
     18    public ChangesetDiscussionTableColumnModel() {
     19        createColumns();
     20    }
     21
    1522    protected void createColumns() {
    16         TableColumn col = null;
    1723        ChangesetDiscussionTableCellRenderer renderer = new ChangesetDiscussionTableCellRenderer();
    1824        // column 0 - Date
    19         col = new TableColumn(0, 150);
     25        TableColumn col = new TableColumn(0, 150);
    2026        col.setHeaderValue(tr("Date"));
    2127        col.setResizable(true);
     
    3945        addColumn(col);
    4046    }
    41 
    42     /**
    43      * Constructs a new {@code ChangesetContentTableColumnModel}.
    44      */
    45     public ChangesetDiscussionTableColumnModel() {
    46         createColumns();
    47     }
    4847}
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableLinkedCellRenderer.java

    r10248 r10308  
    6060        int w = 2;
    6161        int p = 2 + w + 1;
    62         int y1 = 0;
    63         int y2 = 0;
     62        int y1;
     63        int y2;
    6464
    6565        if (value.linkPrev) {
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/sort/WayConnectionTypeCalculator.java

    r10250 r10308  
    145145            RelationMember m, int i, final WayConnectionType wct) {
    146146        Direction dirFW = determineDirection(lastForwardWay, con.get(lastForwardWay).direction, i);
    147         Direction dirBW = NONE;
     147        Direction dirBW;
    148148        if (onewayBeginning) {
    149149            if (lastBackwardWay < 0) {
  • 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}
  • trunk/src/org/openstreetmap/josm/gui/io/LayerNameAndFilePathTableCell.java

    r10021 r10308  
    3232/**
    3333 * Display and edit layer name and file path in a <code>JTable</code>.
    34  * 
     34 *
    3535 * Note: Do not use the same object both as <code>TableCellRenderer</code> and
    3636 * <code>TableCellEditor</code> - this can mess up the current editor component
     
    3939class LayerNameAndFilePathTableCell extends JPanel implements TableCellRenderer, TableCellEditor {
    4040    private static final Color colorError = new Color(255, 197, 197);
    41     private static final String separator = System.getProperty("file.separator");
    42     private static final String ellipsis = '…' + separator;
     41    private static final String ELLIPSIS = '…' + File.separator;
    4342
    4443    private final JLabel lblLayerName = new JLabel();
     
    151150     */
    152151    private String addLblFilename(SaveLayerInfo info) {
    153         String tooltip = "";
     152        String tooltip;
    154153        boolean error = false;
    155154        if (info.getFile() == null) {
     
    185184        while (t != null && !t.isEmpty()) {
    186185            int txtwidth = lblFilename.getFontMetrics(lblFilename.getFont()).stringWidth(t);
    187             if (txtwidth < lblFilename.getWidth() || t.lastIndexOf(separator) < ellipsis.length()) {
     186            if (txtwidth < lblFilename.getWidth() || t.lastIndexOf(File.separator) < ELLIPSIS.length()) {
    188187                break;
    189188            }
    190189            // remove ellipsis, if present
    191             t = hasEllipsis ? t.substring(ellipsis.length()) : t;
     190            t = hasEllipsis ? t.substring(ELLIPSIS.length()) : t;
    192191            // cut next block, and re-add ellipsis
    193             t = ellipsis + t.substring(t.indexOf(separator) + 1);
     192            t = ELLIPSIS + t.substring(t.indexOf(File.separator) + 1);
    194193            hasEllipsis = true;
    195194        }
  • trunk/src/org/openstreetmap/josm/gui/io/SaveLayersTableColumnModel.java

    r9754 r10308  
    8383
    8484    protected void build() {
    85         TableColumn col = null;
    86 
    8785        // column 0 - layer name, save path editor
    8886        LayerNameAndFilePathTableCell lnfpRenderer = new LayerNameAndFilePathTableCell();
    8987        LayerNameAndFilePathTableCell lnfpEditor = new LayerNameAndFilePathTableCell();
    90         col = new TableColumn(0); // keep in sync with SaveLayersModel#columnFilename
     88        TableColumn col = new TableColumn(0); // keep in sync with SaveLayersModel#columnFilename
    9189        col.setHeaderValue(tr("Layer Name and File Path"));
    9290        col.setResizable(true);
  • trunk/src/org/openstreetmap/josm/gui/layer/NoteLayer.java

    r10152 r10308  
    110110            Point p = mv.getPoint(note.getLatLon());
    111111
    112             ImageIcon icon = null;
     112            ImageIcon icon;
    113113            if (note.getId() < 0) {
    114114                icon = NotesDialog.ICON_NEW_SMALL;
  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

    r10234 r10308  
    721721        gbc = GBC.eol();
    722722        gbc.gridx = 0;
    723         gbc.gridy = y++;
     723        gbc.gridy = y;
    724724        panelTf.add(cbShowThumbs, gbc);
    725725
     
    12491249
    12501250        // The searched index is somewhere in the middle, do a binary search from the beginning
    1251         int curIndex = 0;
     1251        int curIndex;
    12521252        int startIndex = 0;
    12531253        int endIndex = lstSize-1;
  • trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyles.java

    r9341 r10308  
    281281                    if (!hasIndependentElemStyle && !multipolygon.getOuterWays().isEmpty()) {
    282282                        Color mpColor = null;
    283                         StyleElementList mpElemStyles = null;
     283                        StyleElementList mpElemStyles;
    284284                        synchronized (ref) {
    285285                            mpElemStyles = get(ref, scale, nc);
  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/Instruction.java

    r8846 r10308  
    7171        @Override
    7272        public void execute(Environment env) {
    73             Object value = null;
     73            Object value;
    7474            if (val instanceof Expression) {
    7575                value = ((Expression) val).evaluate(env);
     
    8888        public String toString() {
    8989            return key + ": " + (val instanceof float[] ? Arrays.toString((float[]) val) :
    90                 val instanceof String ? "String<"+val+'>' : val) + ';';
     90                (val instanceof String ? ("String<"+val+'>') : val)) + ';';
    9191        }
    9292    }
  • trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletingComboBox.java

    r10254 r10308  
    101101
    102102            // item for lookup and selection
    103             Object item = null;
     103            Object item;
    104104            // if the text is a number we don't autocomplete
    105105            if (Main.pref.getBoolean("autocomplete.dont_complete_numbers", true)) {
  • trunk/src/org/openstreetmap/josm/gui/widgets/MultiSplitLayout.java

    r10207 r10308  
    607607            Split split = (Split) root;
    608608            Iterator<Node> splitChildren = split.getChildren().iterator();
    609             Rectangle childBounds = null;
     609            Rectangle childBounds;
    610610            int dividerSize = getDividerSize();
    611611
     
    624624                        splitChildren.hasNext() ? (Divider) (splitChildren.next()) : null;
    625625
    626                     double childWidth = 0.0;
     626                    double childWidth;
    627627                    if (getFloatingDividers()) {
    628628                        childWidth = preferredNodeSize(splitChild).getWidth();
  • trunk/src/org/openstreetmap/josm/io/AbstractReader.java

    r8510 r10308  
    140140            List<RelationMember> relationMembers = new ArrayList<>();
    141141            for (RelationMemberData rm : entry.getValue()) {
    142                 OsmPrimitive primitive = null;
    143 
    144142                // lookup the member from the map of already created primitives
    145                 primitive = externalIdMap.get(new SimplePrimitiveId(rm.getMemberId(), rm.getMemberType()));
     143                OsmPrimitive primitive = externalIdMap.get(new SimplePrimitiveId(rm.getMemberId(), rm.getMemberType()));
    146144
    147145                if (primitive == null) {
  • trunk/src/org/openstreetmap/josm/io/OsmConnection.java

    r10237 r10308  
    1111import java.nio.CharBuffer;
    1212import java.nio.charset.CharacterCodingException;
    13 import java.nio.charset.CharsetEncoder;
    1413import java.nio.charset.StandardCharsets;
    1514import java.util.Objects;
     
    6362     */
    6463    protected void addBasicAuthorizationHeader(HttpClient con) throws OsmTransferException {
    65         CharsetEncoder encoder = StandardCharsets.UTF_8.newEncoder();
    6664        CredentialsAgentResponse response;
    67         String token;
    6865        try {
    6966            synchronized (CredentialsManager.getInstance()) {
     
    7471            throw new OsmTransferException(e);
    7572        }
     73        String token;
    7674        if (response == null) {
    7775            token = ":";
     
    8482            token = username + ':' + password;
    8583            try {
    86                 ByteBuffer bytes = encoder.encode(CharBuffer.wrap(token));
     84                ByteBuffer bytes = StandardCharsets.UTF_8.newEncoder().encode(CharBuffer.wrap(token));
    8785                con.setHeader("Authorization", "Basic "+Base64.encode(bytes));
    8886            } catch (CharacterCodingException e) {
  • trunk/src/org/openstreetmap/josm/io/OsmReader.java

    r10235 r10308  
    316316
    317317    private RelationMemberData parseRelationMember(Relation r) throws XMLStreamException {
    318         String role = null;
    319318        OsmPrimitiveType type = null;
    320319        long id = 0;
     
    339338                    Long.toString(id), Long.toString(r.getUniqueId()), value), e);
    340339        }
    341         value = parser.getAttributeValue(null, "role");
    342         role = value;
     340        String role = parser.getAttributeValue(null, "role");
    343341
    344342        if (id == 0) {
  • trunk/src/org/openstreetmap/josm/io/auth/AbstractCredentialsAgent.java

    r8929 r10308  
    4949                    @Override
    5050                    public void run() {
    51                         CredentialDialog dialog = null;
     51                        CredentialDialog dialog;
    5252                        if (requestorType.equals(RequestorType.PROXY))
    5353                            dialog = CredentialDialog.getHttpProxyCredentialDialog(
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/AddTagsDialog.java

    r10242 r10308  
    272272                @Override
    273273                public void run() {
    274                     String[] tags = null;
    275                     tags = Utils.decodeUrl(args.get("addtags")).split("\\|");
    276274                    Set<String> tagSet = new HashSet<>();
    277                     for (String tag : tags) {
     275                    for (String tag : Utils.decodeUrl(args.get("addtags")).split("\\|")) {
    278276                        if (!tag.trim().isEmpty() && tag.contains("=")) {
    279277                            tagSet.add(tag.trim());
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RemoteControlHttpsServer.java

    r10212 r10308  
    216216    public static Path setupJosmKeystore() throws IOException, GeneralSecurityException {
    217217
    218         char[] storePassword = KEYSTORE_PASSWORD.get().toCharArray();
    219         char[] entryPassword = KEYENTRY_PASSWORD.get().toCharArray();
    220 
    221218        Path dir = Paths.get(RemoteControl.getRemoteControlDir());
    222219        Path path = dir.resolve(KEYSTORE_FILENAME);
     
    249246            KEYENTRY_PASSWORD.put(new BigInteger(130, random).toString(32));
    250247
    251             storePassword = KEYSTORE_PASSWORD.get().toCharArray();
    252             entryPassword = KEYENTRY_PASSWORD.get().toCharArray();
     248            char[] storePassword = KEYSTORE_PASSWORD.get().toCharArray();
     249            char[] entryPassword = KEYENTRY_PASSWORD.get().toCharArray();
    253250
    254251            ks.setKeyEntry(ENTRY_ALIAS, pair.getPrivate(), entryPassword, new Certificate[]{cert});
  • trunk/src/org/openstreetmap/josm/tools/AudioPlayer.java

    r10216 r10308  
    302302                                if (audioInputStream != null) {
    303303                                    Utils.close(audioInputStream);
    304                                     audioInputStream = null;
    305304                                }
    306305                                playingUrl = command.url();
    307306                                audioInputStream = AudioSystem.getAudioInputStream(playingUrl);
    308307                                audioFormat = audioInputStream.getFormat();
    309                                 long nBytesRead = 0;
     308                                long nBytesRead;
    310309                                position = 0.0;
    311310                                offset -= leadIn;
  • trunk/src/org/openstreetmap/josm/tools/Diff.java

    r10246 r10308  
    116116    private final FileData[] filevec;
    117117    private int cost;
    118     /** Snakes bigger than this are considered "big". */
    119     private static final int SNAKE_LIMIT = 20;
    120118
    121119    /**
     
    168166        for (int c = 1;; ++c) {
    169167            int d;          /* Active diagonal. */
    170             boolean bigSnake = false;
    171168
    172169            /* Extend the top-down search by an edit step in each diagonal. */
     
    182179            }
    183180            for (d = fmax; d >= fmin; d -= 2) {
    184                 int x, y, oldx, tlo = fd[fdiagoff + d - 1], thi = fd[fdiagoff + d + 1];
     181                int x, y, tlo = fd[fdiagoff + d - 1], thi = fd[fdiagoff + d + 1];
    185182
    186183                if (tlo >= thi) {
     
    189186                    x = thi;
    190187                }
    191                 oldx = x;
    192188                y = x - d;
    193189                while (x < xlim && y < ylim && xv[x] == yv[y]) {
    194190                    ++x; ++y;
    195                 }
    196                 if (x - oldx > SNAKE_LIMIT) {
    197                     bigSnake = true;
    198191                }
    199192                fd[fdiagoff + d] = x;
     
    216209            }
    217210            for (d = bmax; d >= bmin; d -= 2) {
    218                 int x, y, oldx, tlo = bd[bdiagoff + d - 1], thi = bd[bdiagoff + d + 1];
     211                int x, y, tlo = bd[bdiagoff + d - 1], thi = bd[bdiagoff + d + 1];
    219212
    220213                if (tlo < thi) {
     
    223216                    x = thi - 1;
    224217                }
    225                 oldx = x;
    226218                y = x - d;
    227219                while (x > xoff && y > yoff && xv[x - 1] == yv[y - 1]) {
    228220                    --x; --y;
    229                 }
    230                 if (oldx - x > SNAKE_LIMIT) {
    231                     bigSnake = true;
    232221                }
    233222                bd[bdiagoff + d] = x;
  • trunk/src/org/openstreetmap/josm/tools/ImageProvider.java

    r10300 r10308  
    874874            if ("image/svg+xml".equals(mediatype)) {
    875875                String s = new String(bytes, StandardCharsets.UTF_8);
    876                 SVGDiagram svg = null;
     876                SVGDiagram svg;
    877877                synchronized (getSvgUniverse()) {
    878878                    URI uri = getSvgUniverse().loadSVG(new StringReader(s), Utils.encodeUrl(s));
     
    10031003        switch (type) {
    10041004        case SVG:
    1005             SVGDiagram svg = null;
     1005            SVGDiagram svg;
    10061006            synchronized (getSvgUniverse()) {
    10071007                URI uri = getSvgUniverse().loadSVG(path);
     
    10491049
    10501050    private static URL getImageUrl(String imageName, Collection<String> dirs, Collection<ClassLoader> additionalClassLoaders) {
    1051         URL u = null;
     1051        URL u;
    10521052
    10531053        // Try passed directories first
     
    12251225        }
    12261226
    1227         ImageResource imageResource = null;
     1227        ImageResource imageResource;
    12281228
    12291229        synchronized (ROTATE_CACHE) {
  • trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java

    r10173 r10308  
    237237            if (dpkg || rpm || eque) {
    238238                for (String packageName : packageNames) {
    239                     String[] args = null;
     239                    String[] args;
    240240                    if (dpkg) {
    241241                        args = new String[] {"dpkg-query", "--show", "--showformat", "${Architecture}-${Version}", packageName};
  • trunk/src/org/openstreetmap/josm/tools/TextTagParser.java

    r9078 r10308  
    175175         Pattern p = Pattern.compile(tagRegex);
    176176         Map<String, String> tags = new HashMap<>();
    177          String k = null, v = null;
     177         String k;
     178         String v;
    178179         for (String  line: lines) {
    179180            if (line.trim().isEmpty()) continue; // skip empty lines
Note: See TracChangeset for help on using the changeset viewer.