Changeset 6093 in josm


Ignore:
Timestamp:
2013-07-30T18:35:47+02:00 (11 years ago)
Author:
akks
Message:

see #8902 - collection size ==/!= 0 -> isEmpty()/!isEmpty() (patch by shinigami)

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/AlignInLineAction.java

    r6084 r6093  
    114114
    115115        /// Only ways selected -> Align their nodes.
    116         if ((selectedNodes.size() == 0) && (selectedWays.size() == 1)) { // TODO: handle multiple ways
     116        if ((selectedNodes.isEmpty()) && (selectedWays.size() == 1)) { // TODO: handle multiple ways
    117117            for (Way way : selectedWays) {
    118118                nodes.addAll(way.getNodes());

  • trunk/src/org/openstreetmap/josm/actions/CreateCircleAction.java

    r6084 r6093  
    9898        // special case if no single nodes are selected and exactly one way is:
    9999        // then use the way's nodes
    100         if ((nodes.size() == 0) && (sel.size() == 1)) {
     100        if (nodes.isEmpty() && (sel.size() == 1)) {
    101101            for (OsmPrimitive osm : sel)
    102102                if (osm instanceof Way) {

  • trunk/src/org/openstreetmap/josm/actions/CreateMultipolygonAction.java

    r6084 r6093  
    266266            }
    267267
    268             if( affectedWays.size() > 0 ) {
     268            if(!affectedWays.isEmpty()) {
    269269                // reset key tag on affected ways
    270270                commands.add(new ChangePropertyCommand(affectedWays, key, null));

  • trunk/src/org/openstreetmap/josm/actions/DistributeAction.java

    r6084 r6093  
    6060        // special case if no single nodes are selected and exactly one way is:
    6161        // then use the way's nodes
    62         if ((nodes.size() == 0) && (sel.size() == 1)) {
     62        if (nodes.isEmpty() && (sel.size() == 1)) {
    6363            for (OsmPrimitive osm : sel)
    6464                if (osm instanceof Way) {
    … …  
    122122        // Current number of node
    123123        int pos = 0;
    124         while (nodes.size() > 0) {
     124        while (!nodes.isEmpty()) {
    125125            pos++;
    126126            Node s = null;

  • trunk/src/org/openstreetmap/josm/actions/FollowLineAction.java

    r6069 r6093  
    9696            }
    9797            Set<Node> points = toFollow.getNeighbours(last);
    98             if (!points.remove(prev) || (points.size() == 0))
     98            if (!points.remove(prev) || points.isEmpty())
    9999                continue;
    100100            if (points.size() > 1)    // Ambiguous junction?

  • trunk/src/org/openstreetmap/josm/actions/JoinAreasAction.java

    r6049 r6093  
    198198
    199199        public boolean hasWays() {
    200             return availableWays.size() > 0;
     200            return !availableWays.isEmpty();
    201201        }
    202202
    … …  
    404404        //find intersection points
    405405        Set<Node> nodes = Geometry.addIntersections(allStartingWays, true, cmds);
    406         return nodes.size() > 0;
     406        return !nodes.isEmpty();
    407407    }
    408408
    … …  
    454454
    455455        // Don't warn now, because it will really look corrupted
    456         boolean warnAboutRelations = relations.size() > 0 && allStartingWays.size() > 1;
     456        boolean warnAboutRelations = !relations.isEmpty() && allStartingWays.size() > 1;
    457457
    458458        ArrayList<WayInPolygon> preparedWays = new ArrayList<WayInPolygon>();
    … …  
    507507
    508508        // Delete the discarded inner ways
    509         if (discardedWays.size() > 0) {
     509        if (!discardedWays.isEmpty()) {
    510510            Command deleteCmd = DeleteCommand.delete(Main.map.mapView.getEditLayer(), discardedWays, true);
    511511            if (deleteCmd != null) {
    … …  
    923923
    924924            //process inner ways
    925             if (innerCandidates.size() > 0) {
     925            if (!innerCandidates.isEmpty()) {
    926926                List<PolygonLevel> innerList = findOuterWaysImpl(level + 1, innerCandidates);
    927927                result.addAll(innerList);

  • trunk/src/org/openstreetmap/josm/actions/JoinNodeWayAction.java

    r6084 r6093  
    4747        // If the user has selected some ways, only join the node to these.
    4848        boolean restrictToSelectedWays =
    49             getCurrentDataSet().getSelectedWays().size() > 0;
     49                !getCurrentDataSet().getSelectedWays().isEmpty();
    5050
    5151            List<WaySegment> wss = Main.map.mapView.getNearestWaySegments(
    … …  
    7474            for (Map.Entry<Way, List<Integer>> insertPoint : insertPoints.entrySet()) {
    7575                List<Integer> is = insertPoint.getValue();
    76                 if (is.size() == 0) {
     76                if (is.isEmpty()) {
    7777                    continue;
    7878                }
    … …  
    8888                cmds.add(new ChangeCommand(w, wnew));
    8989            }
    90             if (cmds.size() == 0) return;
     90            if (cmds.isEmpty()) return;
    9191            Main.main.undoRedo.add(new SequenceCommand(tr("Join Node and Line"), cmds));
    9292            Main.map.repaint();

  • trunk/src/org/openstreetmap/josm/actions/MirrorAction.java

    r6084 r6093  
    5151        }
    5252
    53         if (nodes.size() == 0) {
     53        if (nodes.isEmpty()) {
    5454            JOptionPane.showMessageDialog(
    5555                    Main.parent,

  • trunk/src/org/openstreetmap/josm/actions/OrthogonalizeAction.java

    r6084 r6093  
    108108                    }
    109109                }
    110                 if (commands.size() > 0) {
     110                if (!commands.isEmpty()) {
    111111                    Main.main.undoRedo.add(new SequenceCommand(tr("Orthogonalize / Undo"), commands));
    112112                    Main.map.repaint();

  • trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

    r6085 r6093  
    517517
    518518        // don't draw lines if shift is held
    519         if (selection.size() > 0 && !shift) {
     519        if (!selection.isEmpty() && !shift) {
    520520            Node selectedNode = null;
    521521            Way selectedWay = null;
    … …  
    12341234         * Handle special case: Self-Overlapping or closing way
    12351235         */
    1236         if (getCurrentDataSet() != null && getCurrentDataSet().getSelectedWays().size() > 0 && !wayIsFinished && !alt) {
     1236        if (getCurrentDataSet() != null && !getCurrentDataSet().getSelectedWays().isEmpty() && !wayIsFinished && !alt) {
    12371237            Way w = getCurrentDataSet().getSelectedWays().iterator().next();
    12381238            for (Node m : w.getNodes()) {

  • trunk/src/org/openstreetmap/josm/actions/relation/DuplicateRelationAction.java

    r5821 r6093  
    4040    @Override
    4141    public void actionPerformed(ActionEvent e) {
    42         if (!isEnabled() || relations.size()==0)
     42        if (!isEnabled() || relations.isEmpty())
    4343            return;
    4444        Relation r = relations.iterator().next();

  • trunk/src/org/openstreetmap/josm/data/imagery/ImageryLayerInfo.java

    r6087 r6093  
    4242
    4343    public void load() {
    44         boolean addedDefault = layers.size() != 0;
     44        boolean addedDefault = !layers.isEmpty();
    4545        List<ImageryPreferenceEntry> entries = Main.pref.getListOfStructs("imagery.entries", null, ImageryPreferenceEntry.class);
    4646        if (entries != null) {
    … …  
    121121
    122122        Collections.sort(defaultLayers);
    123         Main.pref.putCollection("imagery.layers.default", defaultsSave.size() > 0
    124                 ? defaultsSave : defaults);
     123        Main.pref.putCollection("imagery.layers.default", defaultsSave.isEmpty() ? defaults : defaultsSave);
    125124    }
    126125

  • trunk/src/org/openstreetmap/josm/data/imagery/WmsCache.java

    r5874 r6093  
    269269        index.setTotalFileSize(totalFileSize);
    270270        for (ProjectionEntries projectionEntries: entries.values()) {
    271             if (projectionEntries.entries.size() > 0) {
     271            if (!projectionEntries.entries.isEmpty()) {
    272272                ProjectionType projectionType = new ProjectionType();
    273273                projectionType.setName(projectionEntries.projection);

  • trunk/src/org/openstreetmap/josm/data/osm/MultipolygonCreate.java

    r5909 r6093  
    256256
    257257            //process inner ways
    258             if (innerCandidates.size() > 0) {
     258            if (!innerCandidates.isEmpty()) {
    259259                List<PolygonLevel> innerList = this.findOuterWaysRecursive(level + 1, innerCandidates);
    260260                if (innerList == null) {

  • trunk/src/org/openstreetmap/josm/data/osm/QuadBuckets.java

    r6084 r6093  
    175175                return false;
    176176            boolean ret = this.content.remove(o);
    177             if (this.content.size() == 0) {
     177            if (this.content.isEmpty()) {
    178178                this.content = null;
    179179            }
    … …  
    492492        boolean canRemove()
    493493        {
    494             if (content != null && content.size() > 0)
     494            if (content != null && !content.isEmpty())
    495495                return false;
    496496            if (this.hasChildren())

  • trunk/src/org/openstreetmap/josm/data/projection/datum/NTV2GridShiftFile.java

    r5874 r6093  
    184184        for (int i = 0; i < subGrid.length; i++) {
    185185            ArrayList<NTV2SubGrid> subSubGrids = subGridMap.get(subGrid[i].getSubGridName());
    186             if (subSubGrids.size() > 0) {
     186            if (!subSubGrids.isEmpty()) {
    187187                NTV2SubGrid[] subGridArray = subSubGrids.toArray(nullArray);
    188188                subGrid[i].setSubGridArray(subGridArray);

  • trunk/src/org/openstreetmap/josm/data/validation/tests/Coastlines.java

    r4869 r6093  
    157157                }
    158158
    159                 if (highlight.size() > 0) {
     159                if (!highlight.isEmpty()) {
    160160                    errors.add(new TestError(this, Severity.ERROR, tr("Unconnected coastline"),
    161161                            UNCONNECTED_COASTLINE, primitives, highlight));

  • trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java

    r6068 r6093  
    109109            }
    110110        }
    111         if (allroles.size() == 0) {
     111        if (allroles.isEmpty()) {
    112112            errors.add( new TestError(this, Severity.WARNING, tr("Relation type is unknown"),
    113113                    RELATION_UNKNOWN, n) );

  • trunk/src/org/openstreetmap/josm/gui/MultiSplitLayout.java

    r6084 r6093  
    11931193    private static void addSplitChild(Split parent, Node child) {
    11941194        List<Node> children = new ArrayList<Node>(parent.getChildren());
    1195         if (children.size() == 0) {
     1195        if (children.isEmpty()) {
    11961196            children.add(child);
    11971197        }

  • trunk/src/org/openstreetmap/josm/gui/NavigatableComponent.java

    r6070 r6093  
    15391539     */
    15401540    public void setNewCursor(Cursor cursor, Object reference) {
    1541         if(Cursors.size() > 0) {
     1541        if(!Cursors.isEmpty()) {
    15421542            CursorInfo l = Cursors.getLast();
    15431543            if(l != null && l.cursor == cursor && l.object == reference)
    … …  
    15551555     */
    15561556    public void resetCursor(Object reference) {
    1557         if(Cursors.size() == 0) {
     1557        if(Cursors.isEmpty()) {
    15581558            setCursor(null);
    15591559            return;
    … …  
    15621562        stripCursors(reference);
    15631563        if(l != null && l.object == reference) {
    1564             if(Cursors.size() == 0) {
     1564            if(Cursors.isEmpty()) {
    15651565                setCursor(null);
    15661566            } else {

  • trunk/src/org/openstreetmap/josm/gui/conflict/tags/CombinePrimitiveResolverDialog.java

    r6084 r6093  
    259259
    260260        TagCollection allResolutions = getTagConflictResolverModel().getAllResolutions();
    261         if (allResolutions.size() > 0) {
     261        if (!allResolutions.isEmpty()) {
    262262            cmds.addAll(buildTagChangeCommand(targetPrimitive, allResolutions));
    263263        }

  • trunk/src/org/openstreetmap/josm/gui/conflict/tags/TagConflictResolutionUtil.java

    r5700 r6093  
    3939        int numNodesWithTags = 0;
    4040        for (OsmPrimitive p: merged) {
    41             if (p.getKeys().size() >0) {
     41            if (!p.getKeys().isEmpty()) {
    4242                numNodesWithTags++;
    4343            }

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

    r6084 r6093  
    280280
    281281    private void setOkEnabled(boolean b) {
    282         if (buttons != null && buttons.size() > 0) {
     282        if (buttons != null && !buttons.isEmpty()) {
    283283            buttons.get(0).setEnabled(b);
    284284        }

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

    r6085 r6093  
    841841        }
    842842        add(data, BorderLayout.CENTER);
    843         if (buttons.size() > 0 && buttons.get(0) != null && !buttons.get(0).isEmpty()) {
     843        if (!buttons.isEmpty() && buttons.get(0) != null && !buttons.get(0).isEmpty()) {
    844844            buttonsPanel = new JPanel(new GridLayout(buttons.size(), 1));
    845845            for (Collection<SideButton> buttonRow : buttons) {

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/DownloadRelationMemberTask.java

    r6084 r6093  
    9191
    9292    protected String buildDownloadFeedbackMessage() {
    93         if (parents.size() == 0) {
     93        if (parents.isEmpty()) {
    9494            return trn("Downloading {0} incomplete object",
    9595                    "Downloading {0} incomplete objects",

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableModel.java

    r6084 r6093  
    551551        // make the first selected member visible
    552552        //
    553         if (selectedIndices.size() > 0) {
     553        if (!selectedIndices.isEmpty()) {
    554554            fireMakeMemberVisible(Collections.min(selectedIndices));
    555555        }
    … …  
    630630        }
    631631        getSelectionModel().setValueIsAdjusting(false);
    632         if (getSelectedIndices().size() > 0) {
     632        if (!getSelectedIndices().isEmpty()) {
    633633            fireMakeMemberVisible(getSelectedIndices().get(0));
    634634        }

  • trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java

    r6085 r6093  
    183183        }
    184184
    185         if (data.tracks.size() > 0) {
     185        if (!data.tracks.isEmpty()) {
    186186            info.append("<table><thead align='center'><tr><td colspan='5'>"
    187187                    + trn("{0} track", "{0} tracks", data.tracks.size(), data.tracks.size())

  • trunk/src/org/openstreetmap/josm/gui/layer/TMSLayer.java

    r6085 r6093  
    13871387                // Instantiating large TileSets is expensive.  If there
    13881388                // are no loaded tiles, don't bother even trying.
    1389                 if (ts2.allLoadedTiles().size() == 0) {
     1389                if (ts2.allLoadedTiles().isEmpty()) {
    13901390                    newlyMissedTiles.add(missed);
    13911391                    continue;

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

    r6090 r6093  
    470470        }
    471471
    472         if (gpxLst.size() == 0) {
     472        if (gpxLst.isEmpty()) {
    473473            gpxLst.add(new GpxDataWrapper(tr("<No GPX track loaded yet>"), null, null));
    474474        }

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/GeoImageLayer.java

    r6084 r6093  
    239239                Main.main.addLayer(layer);
    240240
    241                 if (! canceled && layer.data.size() > 0) {
     241                if (!canceled && !layer.data.isEmpty()) {
    242242                    boolean noGeotagFound = true;
    243243                    for (ImageEntry e : layer.data) {
    … …  
    633633
    634634    public void showPreviousPhoto() {
    635         if (data != null && data.size() > 0) {
     635        if (data != null && !data.isEmpty()) {
    636636            currentPhoto--;
    637637            if (currentPhoto < 0) {

  • trunk/src/org/openstreetmap/josm/gui/layer/gpx/MarkersFromNamedPointsAction.java

    r6070 r6093  
    4040        }
    4141        MarkerLayer ml = new MarkerLayer(namedTrackPoints, tr("Named Trackpoints from {0}", layer.getName()), layer.getAssociatedFile(), layer);
    42         if (ml.data.size() > 0) {
     42        if (!ml.data.isEmpty()) {
    4343            Main.main.addLayer(ml);
    4444        }

  • trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPreset.java

    r6074 r6093  
    215215        }
    216216        p.add(items, GBC.eol().fill());
    217         if (selected.size() == 0 && !supportsRelation()) {
     217        if (selected.isEmpty() && !supportsRelation()) {
    218218            GuiHelper.setEnabledRec(items, false);
    219219        }
    … …  
    331331            }
    332332
    333             answer = new PresetDialog(p, title, (ImageIcon) getValue(Action.SMALL_ICON), (sel.size() == 0)).getValue();
     333            answer = new PresetDialog(p, title, (ImageIcon) getValue(Action.SMALL_ICON), sel.isEmpty()).getValue();
    334334        }
    335335        if (!showNewRelation && answer == 2)
    … …  
    350350     */
    351351    public Collection<OsmPrimitive> createSelection(Collection<OsmPrimitive> participants) {
    352         originalSelectionEmpty = participants.size() == 0;
     352        originalSelectionEmpty = participants.isEmpty();
    353353        Collection<OsmPrimitive> sel = new LinkedList<OsmPrimitive>();
    354354        for (OsmPrimitive osm : participants)

  • trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPresetItems.java

    r6087 r6093  
    358358        public boolean addToPanel(JPanel p, Collection<OsmPrimitive> sel) {
    359359            p.add(new JLabel(" "), GBC.eol()); // space
    360             if (roles.size() > 0) {
     360            if (!roles.isEmpty()) {
    361361                JPanel proles = new JPanel(new GridBagLayout());
    362362                proles.add(new JLabel(tr("Available roles")), GBC.std().insets(0, 0, 10, 0));

  • trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPresetReader.java

    r6074 r6093  
    101101                lastrole = null;
    102102            } else {
    103                 if (all.size() != 0) {
     103                if (!all.isEmpty()) {
    104104                    if (o instanceof TaggingPresetItems.Roles) {
    105105                        all.getLast().data.add((TaggingPresetItem) o);

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

    r6084 r6093  
    4141                    if (Main.pref.getBoolean("marker.makeautomarkers", true)) {
    4242                        MarkerLayer ml = new MarkerLayer(r.data, tr("Markers from {0}", fn), fileFinal, gpxLayer);
    43                         if (ml.data.size() > 0) {
     43                        if (!ml.data.isEmpty()) {
    4444                            Main.main.addLayer(ml);
    4545                        }

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

    r6070 r6093  
    232232            }
    233233        }
    234         if (w.isDeleted() && nodeIds.size() > 0) {
     234        if (w.isDeleted() && !nodeIds.isEmpty()) {
    235235            System.out.println(tr("Deleted way {0} contains nodes", w.getUniqueId()));
    236236            nodeIds = new ArrayList<Long>();
    … …  
    279279            }
    280280        }
    281         if (r.isDeleted() && members.size() > 0) {
     281        if (r.isDeleted() && !members.isEmpty()) {
    282282            System.out.println(tr("Deleted relation {0} contains members", r.getUniqueId()));
    283283            members = new ArrayList<RelationMemberData>();

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

    r6070 r6093  
    8080        GpxLayer gpxLayer = null;
    8181        List<SessionReader.LayerDependency> deps = support.getLayerDependencies();
    82         if (deps.size() > 0) {
     82        if (!deps.isEmpty()) {
    8383            Layer layer = deps.iterator().next().getLayer();
    8484            if (layer instanceof GpxLayer) {

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

    r6087 r6093  
    4848            GpxLayer gpxLayer = null;
    4949            List<SessionReader.LayerDependency> deps = support.getLayerDependencies();
    50             if (deps.size() > 0) {
     50            if (!deps.isEmpty()) {
    5151                Layer layer = deps.iterator().next().getLayer();
    5252                if (layer instanceof GpxLayer) {

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

    r5874 r6093  
    197197            }
    198198            Set<Layer> deps = dependencies.get(layer);
    199             if (deps.size() > 0) {
     199            if (!deps.isEmpty()) {
    200200                List<Integer> depsInt = new ArrayList<Integer>();
    201201                for (Layer depLayer : deps) {

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

    r6070 r6093  
    173173                            }
    174174                        }
    175                         else if (test && intersectionNodes.size() > 0)
     175                        else if (test && !intersectionNodes.isEmpty())
    176176                            return intersectionNodes;
    177177                    }

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

    r6087 r6093  
    425425                    String cache_name = full_name;
    426426                    /* cache separately */
    427                     if (dirs != null && dirs.size() > 0) {
     427                    if (dirs != null && !dirs.isEmpty()) {
    428428                        cache_name = "id:" + id + ":" + full_name;
    429429                        if(archive != null) {

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

    r6070 r6093  
    443443            T parentless = null;
    444444            for (T key : deps.keySet()) {
    445                 if (deps.get(key).size() == 0) {
     445                if (deps.get(key).isEmpty()) {
    446446                    parentless = key;
    447447                    break;
Note: See TracChangeset for help on using the changeset viewer.