Changeset 34653 in osm


Ignore:
Timestamp:
2018-09-15T14:06:08+02:00 (6 years ago)
Author:
donvip
Message:

fix UnnecessaryParentheses warnings

Location:
applications/editors/josm/plugins/public_transport/src/org/openstreetmap/josm/plugins/public_transport
Files:
8 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/public_transport/src/org/openstreetmap/josm/plugins/public_transport/PublicTransportAStar.java

    r33765 r34653  
    2929        @Override
    3030        public boolean equals(Object o) {
    31             if ((NodeVertex) o == null)
     31            if (!(o instanceof NodeVertex))
    3232                return false;
    3333            return node.equals(((NodeVertex) o).node);

  • applications/editors/josm/plugins/public_transport/src/org/openstreetmap/josm/plugins/public_transport/actions/GTFSImporterAction.java

    r34548 r34653  
    296296
    297297        for (int i = 0; i < table.getRowCount(); ++i) {
    298             if ((nodes.elementAt(i) != null) && (ds.isSelected(nodes.elementAt(i))))
     298            if (nodes.elementAt(i) != null && ds.isSelected(nodes.elementAt(i)))
    299299                table.addRowSelectionInterval(i, i);
    300300        }

  • applications/editors/josm/plugins/public_transport/src/org/openstreetmap/josm/plugins/public_transport/actions/RoutePatternAction.java

    r34548 r34653  
    8888        @Override
    8989        public boolean isCellEditable(int row, int column) {
    90             if ((column == 0) && (hasFixedKeys))
     90            if (column == 0 && hasFixedKeys)
    9191                return false;
    9292            return true;
    … …  
    136136                if (key == null)
    137137                    return;
    138                 if ((blacklist == null) || (!blacklist.contains(key))) {
     138                if (blacklist == null || !blacklist.contains(key)) {
    139139                    relation.setModified(true);
    140140                    if ("".equals(getValueAt(e.getFirstRow(), 1)))
    … …  
    212212                buf[0] = curName;
    213213            } else {
    214                 buf[0] = tr("[ID] {0}", (Long.valueOf(node.getId())).toString());
     214                buf[0] = tr("[ID] {0}", Long.valueOf(node.getId()).toString());
    215215            }
    216216            String curRef = node.get("ref");
    … …  
    10821082
    10831083            for (int i = 0; i < itineraryData.getRowCount(); ++i) {
    1084                 if ((itineraryData.ways.elementAt(i) != null)
    1085                         && (mainDataSet.isSelected(itineraryData.ways.elementAt(i))))
     1084                if (itineraryData.ways.elementAt(i) != null
     1085                        && mainDataSet.isSelected(itineraryData.ways.elementAt(i)))
    10861086                    itineraryTable.addRowSelectionInterval(i, i);
    10871087            }
    … …  
    10901090            if (itineraryTable.getSelectedRowCount() > 0) {
    10911091                for (int i = 0; i < itineraryData.getRowCount(); ++i) {
    1092                     if ((itineraryTable.isRowSelected(i))
    1093                             && (itineraryData.ways.elementAt(i) != null)) {
     1092                    if (itineraryTable.isRowSelected(i)
     1093                            && itineraryData.ways.elementAt(i) != null) {
    10941094                        itineraryData.ways.elementAt(i).accept((PrimitiveVisitor) box);
    10951095                    }
    … …  
    11121112            if (itineraryTable.getSelectedRowCount() > 0) {
    11131113                for (int i = 0; i < itineraryData.getRowCount(); ++i) {
    1114                     if ((itineraryTable.isRowSelected(i))
    1115                             && (itineraryData.ways.elementAt(i) != null)) {
     1114                    if (itineraryTable.isRowSelected(i)
     1115                            && itineraryData.ways.elementAt(i) != null) {
    11161116                        mainDataSet.addSelected(itineraryData.ways.elementAt(i));
    11171117
    11181118                        RelationMember markedWay = new RelationMember(
    1119                                 (String) (itineraryData.getValueAt(i, 1)),
     1119                                (String) itineraryData.getValueAt(i, 1),
    11201120                                itineraryData.ways.elementAt(i));
    11211121                        markedWays.addElement(markedWay);
    … …  
    11281128
    11291129                        RelationMember markedWay = new RelationMember(
    1130                                 (String) (itineraryData.getValueAt(i, 1)),
     1130                                (String) itineraryData.getValueAt(i, 1),
    11311131                                itineraryData.ways.elementAt(i));
    11321132                        markedWays.addElement(markedWay);
    … …  
    11431143            while (relIter.hasNext()) {
    11441144                RelationMember curMember = relIter.next();
    1145                 if ((curMember.isWay()) && (mainDataSet.isSelected(curMember.getWay()))) {
     1145                if (curMember.isWay() && mainDataSet.isSelected(curMember.getWay())) {
    11461146                    itineraryData.insertRow(insPos, curMember.getWay(), curMember.getRole());
    11471147                    if (insPos >= 0)
    … …  
    11571157            while (wayIter.hasNext()) {
    11581158                Way curMember = wayIter.next();
    1159                 if (!(addedWays.contains(curMember))) {
     1159                if (!addedWays.contains(curMember)) {
    11601160                    itineraryData.insertRow(insPos, curMember, "");
    11611161                    if (insPos >= 0)
    … …  
    11791179        } else if ("routePattern.itineraryDelete".equals(event.getActionCommand())) {
    11801180            for (int i = itineraryData.getRowCount() - 1; i >= 0; --i) {
    1181                 if ((itineraryTable.isRowSelected(i))
    1182                         && (itineraryData.ways.elementAt(i) != null)) {
     1181                if (itineraryTable.isRowSelected(i)
     1182                        && itineraryData.ways.elementAt(i) != null) {
    11831183                    itineraryData.ways.removeElementAt(i);
    11841184                    itineraryData.removeRow(i);
    … …  
    11981198            if (itineraryTable.getSelectedRowCount() > 0) {
    11991199                for (int i = itineraryData.getRowCount() - 1; i >= 0; --i) {
    1200                     if ((itineraryTable.isRowSelected(i))
    1201                             && (itineraryData.ways.elementAt(i) != null)) {
    1202                         if (!(usedWays.contains(itineraryData.ways.elementAt(i)))) {
     1200                    if (itineraryTable.isRowSelected(i)
     1201                            && itineraryData.ways.elementAt(i) != null) {
     1202                        if (!usedWays.contains(itineraryData.ways.elementAt(i))) {
    12031203                            addWayToSortingData(itineraryData.ways.elementAt(i), frontNodes,
    12041204                                    backNodes, loops);
    … …  
    12131213                for (int i = itineraryData.getRowCount() - 1; i >= 0; --i) {
    12141214                    if (itineraryData.ways.elementAt(i) != null) {
    1215                         if (!(usedWays.contains(itineraryData.ways.elementAt(i)))) {
     1215                        if (!usedWays.contains(itineraryData.ways.elementAt(i))) {
    12161216                            addWayToSortingData(itineraryData.ways.elementAt(i), frontNodes,
    12171217                                    backNodes, loops);
    … …  
    12561256            if (itineraryTable.getSelectedRowCount() > 0) {
    12571257                for (int i = itineraryData.getRowCount() - 1; i >= 0; --i) {
    1258                     if ((itineraryTable.isRowSelected(i))
    1259                             && (itineraryData.ways.elementAt(i) != null)) {
    1260                         String role = (String) (itineraryData.getValueAt(i, 1));
     1258                    if (itineraryTable.isRowSelected(i)
     1259                            && itineraryData.ways.elementAt(i) != null) {
     1260                        String role = (String) itineraryData.getValueAt(i, 1);
    12611261                        if ("backward".equals(role))
    12621262                            role = "forward";
    … …  
    12761276                for (int i = itineraryData.getRowCount() - 1; i >= 0; --i) {
    12771277                    if (itineraryData.ways.elementAt(i) != null) {
    1278                         String role = (String) (itineraryData.getValueAt(i, 1));
     1278                        String role = (String) itineraryData.getValueAt(i, 1);
    12791279                        if ("backward".equals(role))
    12801280                            role = "forward";
    … …  
    13151315
    13161316            for (int i = 0; i < stoplistData.getRowCount(); ++i) {
    1317                 if ((stoplistData.nodes.elementAt(i) != null)
    1318                         && (mainDataSet.isSelected(stoplistData.nodes.elementAt(i))))
     1317                if (stoplistData.nodes.elementAt(i) != null
     1318                        && mainDataSet.isSelected(stoplistData.nodes.elementAt(i)))
    13191319                    stoplistTable.addRowSelectionInterval(i, i);
    13201320            }
    … …  
    13461346
    13471347                        RelationMember markedNode = new RelationMember(
    1348                                 (String) (stoplistData.getValueAt(i, 1)),
     1348                                (String) stoplistData.getValueAt(i, 1),
    13491349                                stoplistData.nodes.elementAt(i));
    13501350                        markedNodes.addElement(markedNode);
    … …  
    13561356
    13571357                    RelationMember markedNode = new RelationMember(
    1358                             (String) (stoplistData.getValueAt(i, 1)),
     1358                            (String) stoplistData.getValueAt(i, 1),
    13591359                            stoplistData.nodes.elementAt(i));
    13601360                    markedNodes.addElement(markedNode);
    … …  
    13701370            while (relIter.hasNext()) {
    13711371                RelationMember curMember = relIter.next();
    1372                 if ((curMember.isNode()) && (mainDataSet.isSelected(curMember.getNode()))) {
     1372                if (curMember.isNode() && mainDataSet.isSelected(curMember.getNode())) {
    13731373                    StopReference sr = detectMinDistance(curMember.getNode(), segmentMetrics,
    13741374                            cbRight.isSelected(), cbLeft.isSelected());
    … …  
    13871387            while (nodeIter.hasNext()) {
    13881388                Node curMember = nodeIter.next();
    1389                 if (!(addedNodes.contains(curMember))) {
     1389                if (!addedNodes.contains(curMember)) {
    13901390                    StopReference sr = detectMinDistance(curMember, segmentMetrics,
    13911391                            cbRight.isSelected(), cbLeft.isSelected());
    … …  
    14871487                for (int i = stoplistData.getRowCount() - 1; i >= 0; --i) {
    14881488                    if (stoplistTable.isRowSelected(i)) {
    1489                         String role = (String) (stoplistData.getValueAt(i, STOPLIST_ROLE_COLUMN));
     1489                        String role = (String) stoplistData.getValueAt(i, STOPLIST_ROLE_COLUMN);
    14901490                        RelationMember markedNode = new RelationMember(role,
    14911491                                stoplistData.nodes.elementAt(i));
    … …  
    14981498            } else {
    14991499                for (int i = stoplistData.getRowCount() - 1; i >= 0; --i) {
    1500                     String role = (String) (stoplistData.getValueAt(i, STOPLIST_ROLE_COLUMN));
     1500                    String role = (String) stoplistData.getValueAt(i, STOPLIST_ROLE_COLUMN);
    15011501                    RelationMember markedNode = new RelationMember(role,
    15021502                            stoplistData.nodes.elementAt(i));
    … …  
    16511651            if (itineraryData.ways.elementAt(i) != null) {
    16521652                RelationMember member = new RelationMember(
    1653                         (String) (itineraryData.getValueAt(i, 1)), itineraryData.ways.elementAt(i));
     1653                        (String) itineraryData.getValueAt(i, 1), itineraryData.ways.elementAt(i));
    16541654                members.add(member);
    16551655            }
    … …  
    16681668        for (int i = 0; i < stoplistData.getRowCount(); ++i) {
    16691669            RelationMember member = new RelationMember(
    1670                     (String) (stoplistData.getValueAt(i, STOPLIST_ROLE_COLUMN)),
     1670                    (String) stoplistData.getValueAt(i, STOPLIST_ROLE_COLUMN),
    16711671                    stoplistData.nodes.elementAt(i));
    16721672            currentRoute.addMember(member);
    … …  
    18651865            if (itineraryData.ways.elementAt(i) != null) {
    18661866                Way way = itineraryData.ways.elementAt(i);
    1867                 if (!(way.isIncomplete())) {
    1868                     if ("backward".equals((itineraryData.getValueAt(i, 1)))) {
     1867                if (!way.isIncomplete()) {
     1868                    if ("backward".equals(itineraryData.getValueAt(i, 1))) {
    18691869                        for (int j = way.getNodesCount() - 2; j >= 0; --j) {
    18701870                            SegmentMetric sm = new SegmentMetric(way.getNode(j + 1).getCoor().lat(),
    … …  
    19281928                    boolean vertexOnSeg = (vertexAngle == segAngle)
    19291929                            || (vertexAngle == lastSegAngle);
    1930                     boolean vertexOnTheLeft = (!vertexOnSeg)
     1930                    boolean vertexOnTheLeft = !vertexOnSeg
    19311931                            && (((lastSegAngle > vertexAngle) && (vertexAngle > segAngle))
    19321932                                    || ((vertexAngle > segAngle) && (segAngle > lastSegAngle))
    … …  
    19431943                        curPosition += 2 * Math.PI;
    19441944                    if ((Math.abs(currentDistance) < distance)
    1945                             && (((!vertexOnTheLeft) && (rhsPossible))
    1946                                     || ((vertexOnTheLeft) && (lhsPossible)) || (vertexOnSeg))) {
     1945                            && ((!vertexOnTheLeft && rhsPossible)
     1946                                    || (vertexOnTheLeft && lhsPossible) || vertexOnSeg)) {
    19471947                        distance = Math.abs(currentDistance);
    19481948                        minIndex = curIndex - 1;
    … …  
    19581958                double currentDistance = (lat - sm.aLat) * sm.o1 + (lon - sm.aLon) * sm.o2;
    19591959                if ((Math.abs(currentDistance) < distance)
    1960                         && (((currentDistance >= 0) && (rhsPossible))
    1961                                 || ((currentDistance <= 0) && (lhsPossible)))) {
     1960                        && ((currentDistance >= 0 && rhsPossible)
     1961                                || (currentDistance <= 0 && lhsPossible))) {
    19621962                    distance = Math.abs(currentDistance);
    19631963                    minIndex = curIndex;

  • applications/editors/josm/plugins/public_transport/src/org/openstreetmap/josm/plugins/public_transport/actions/StopImporterAction.java

    r34548 r34653  
    124124            refreshData();
    125125        } else if ("stopImporter.settingsGPSTimeStart".equals(event.getActionCommand())) {
    126             if ((!inEvent) && (dialog.gpsTimeStartValid()) && (currentTrack != null))
     126            if (!inEvent && dialog.gpsTimeStartValid() && currentTrack != null)
    127127                UndoRedoHandler.getInstance().add(new TrackStoplistRelocateCommand(this));
    128128        } else if ("stopImporter.settingsStopwatchStart".equals(event.getActionCommand())) {
    129             if ((!inEvent) && (dialog.stopwatchStartValid()) && (currentTrack != null))
     129            if (!inEvent && dialog.stopwatchStartValid() && currentTrack != null)
    130130                UndoRedoHandler.getInstance().add(new TrackStoplistRelocateCommand(this));
    131131        } else if ("stopImporter.settingsTimeWindow".equals(event.getActionCommand())) {
    … …  
    239239    public void tracksSelectionChanged(int selectedPos) {
    240240        if (selectedPos >= 0) {
    241             currentTrack = (tracksListModel.elementAt(selectedPos));
     241            currentTrack = tracksListModel.elementAt(selectedPos);
    242242            dialog.setTrackValid(true);
    243243
    … …  
    328328
    329329        for (int i = 0; i < table.getRowCount(); ++i) {
    330             if ((nodes.elementAt(i) != null) && (ds.isSelected(nodes.elementAt(i))))
     330            if (nodes.elementAt(i) != null && ds.isSelected(nodes.elementAt(i)))
    331331                table.addRowSelectionInterval(i, i);
    332332        }

  • applications/editors/josm/plugins/public_transport/src/org/openstreetmap/josm/plugins/public_transport/commands/AbstractGTFSCatchJoinCommand.java

    r33817 r34653  
    5353        while (iter.hasNext()) {
    5454            Node n = iter.next();
    55             if ((n != null) && (n.equals(gtfsStopTM.nodes.elementAt(j))))
     55            if (n != null && n.equals(gtfsStopTM.nodes.elementAt(j)))
    5656                continue;
    5757            if (dest != null)

  • applications/editors/josm/plugins/public_transport/src/org/openstreetmap/josm/plugins/public_transport/models/GTFSStopTableModel.java

    r34166 r34653  
    112112            if ('"' == s.charAt(pos))
    113113                insideDoubleQuoted = !insideDoubleQuoted;
    114             else if ((separator == s.charAt(pos)) && (!insideDoubleQuoted))
     114            else if (separator == s.charAt(pos) && !insideDoubleQuoted)
    115115                break;
    116116            ++pos;
    … …  
    187187
    188188        if (insPos == -1) {
    189             if ((nearBusStop) || !(tr("pending").equals(buf[2])))
     189            if (nearBusStop || !tr("pending").equals(buf[2]))
    190190                nodes.addElement(null);
    191191            else {
    … …  
    197197            super.addRow(buf);
    198198        } else {
    199             if ((nearBusStop) || !(tr("pending").equals(buf[2])))
     199            if (nearBusStop || !tr("pending").equals(buf[2]))
    200200                nodes.insertElementAt(null, insPos);
    201201            else {

  • applications/editors/josm/plugins/public_transport/src/org/openstreetmap/josm/plugins/public_transport/models/ItineraryTableModel.java

    r33765 r34653  
    5959            buf[0] = curName;
    6060        else
    61             buf[0] = tr("[ID] {0}", (Long.valueOf(way.getId())).toString());
     61            buf[0] = tr("[ID] {0}", Long.valueOf(way.getId()));
    6262        buf[1] = role;
    6363        if (insPos == -1) {
    … …  
    9494                break;
    9595
    96             boolean gapRequired = gapNecessary(ways.elementAt(i), (String) (getValueAt(i, 1)),
     96            boolean gapRequired = gapNecessary(ways.elementAt(i), (String) getValueAt(i, 1),
    9797                    lastNode);
    98             if ((i > 0) && (!gapRequired) && (ways.elementAt(i - 1) == null)) {
     98            if (i > 0 && !gapRequired && ways.elementAt(i - 1) == null) {
    9999                ways.removeElementAt(i - 1);
    100100                removeRow(i - 1);
    101101                --i;
    102             } else if ((i > 0) && gapRequired && (ways.elementAt(i - 1) != null)) {
     102            } else if (i > 0 && gapRequired && ways.elementAt(i - 1) != null) {
    103103                String[] buf = {"", ""};
    104104                buf[0] = tr("[gap]");
    … …  
    106106                ++i;
    107107            }
    108             lastNode = getLastNode(ways.elementAt(i), (String) (getValueAt(i, 1)));
     108            lastNode = getLastNode(ways.elementAt(i), (String) getValueAt(i, 1));
    109109        }
    110         while ((getRowCount() > 0) && (ways.elementAt(getRowCount() - 1) == null)) {
     110        while (getRowCount() > 0 && ways.elementAt(getRowCount() - 1) == null) {
    111111            ways.removeElementAt(getRowCount() - 1);
    112112            removeRow(getRowCount() - 1);
    … …  
    126126
    127127    private Node getLastNode(Way way, String role) {
    128         if ((way == null) || (way.isIncomplete()) || (way.getNodesCount() < 1))
     128        if (way == null || way.isIncomplete() || way.getNodesCount() < 1)
    129129            return null;
    130130        else {
    … …  
    137137
    138138    private boolean gapNecessary(Way way, String role, Node lastNode) {
    139         if ((way != null) && (!(way.isIncomplete())) && (way.getNodesCount() >= 1)) {
     139        if (way != null && !way.isIncomplete() && way.getNodesCount() >= 1) {
    140140            Node firstNode = null;
    141141            if ("backward".equals(role))
    … …  
    143143            else
    144144                firstNode = way.getNode(0);
    145             if ((lastNode != null) && (!lastNode.equals(firstNode)))
     145            if (lastNode != null && !lastNode.equals(firstNode))
    146146                return true;
    147147        }

  • applications/editors/josm/plugins/public_transport/src/org/openstreetmap/josm/plugins/public_transport/refs/TrackReference.java

    r34548 r34653  
    5151        if (track != null) {
    5252            Iterator<GpxTrackSegment> siter = track.getSegments().iterator();
    53             while ((siter.hasNext()) && (this.gpsSyncTime == null)) {
     53            while (siter.hasNext() && this.gpsSyncTime == null) {
    5454                Iterator<WayPoint> witer = siter.next().getWayPoints().iterator();
    5555                if (witer.hasNext()) {
Note: See TracChangeset for help on using the changeset viewer.