Changeset 6009 in josm for trunk/src/org/openstreetmap/josm


Ignore:
Timestamp:
2013-06-16T13:39:50+02:00 (12 years ago)
Author:
Don-vip
Message:

fix #8799 - Follow conventional visitor design pattern by renaming visit(Visitor) to accept(Visitor)

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

Legend:

Unmodified
Added
Removed

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

    r5958 r6009  
    235235            }
    236236            for (OsmPrimitive osm : sel) {
    237                 osm.visit(v);
     237                osm.accept(v);
    238238            }
    239239            // increase bbox by 0.001 degrees on each side. this is required

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

    r5909 r6009  
    204204                    // we upload modified nodes even if they aren't in the current
    205205                    // selection.
    206                     n.visit(this);
     206                    n.accept(this);
    207207                }
    208208            }
    … …  
    218218                    //
    219219                    if (p.isNewOrUndeleted()) {
    220                         p.visit(this);
     220                        p.accept(this);
    221221                    }
    222222                }
    … …  
    240240            hull = new HashSet<OsmPrimitive>();
    241241            for (OsmPrimitive p: base) {
    242                 p.visit(this);
     242                p.accept(this);
    243243            }
    244244            return hull;

  • trunk/src/org/openstreetmap/josm/command/ChangePropertyKeyCommand.java

    r4918 r6009  
    7272        if (objects.size() == 1) {
    7373            NameVisitor v = new NameVisitor();
    74             objects.iterator().next().visit(v);
     74            objects.iterator().next().accept(v);
    7575            text += " "+tr(v.className)+" "+v.name;
    7676        } else {
    … …  
    9393        final NameVisitor v = new NameVisitor();
    9494        for (final OsmPrimitive osm : objects) {
    95             osm.visit(v);
     95            osm.accept(v);
    9696            children.add(new PseudoCommand() {
    9797                @Override

  • trunk/src/org/openstreetmap/josm/command/Command.java

    r5824 r6009  
    8686        fillModifiedData(all, all, all);
    8787        for (OsmPrimitive osm : all) {
    88             osm.visit(visitor);
     88            osm.accept(visitor);
    8989        }
    9090        cloneMap = visitor.orig;

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

    r5589 r6009  
    3131    void setChangesetId(int changesetId);
    3232
    33     void visit(PrimitiveVisitor visitor);
     33    void accept(PrimitiveVisitor visitor);
    3434    String getName();
    3535    String getLocalName();

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

    r5935 r6009  
    196196
    197197    @Override
    198     public void visit(Visitor visitor) {
     198    public void accept(Visitor visitor) {
    199199        visitor.visit(this);
    200200    }
    201201
    202202    @Override
    203     public void visit(PrimitiveVisitor visitor) {
     203    public void accept(PrimitiveVisitor visitor) {
    204204        visitor.visit(this);
    205205    }

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

    r5170 r6009  
    7373
    7474    @Override
    75     public void visit(PrimitiveVisitor visitor) {
     75    public void accept(PrimitiveVisitor visitor) {
    7676        visitor.visit(this);
    7777    }

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

    r5754 r6009  
    981981            OsmPrimitive ref = (OsmPrimitive) this.referrers;
    982982            if (ref.dataSet == dataSet) {
    983                 ref.visit(visitor);
     983                ref.accept(visitor);
    984984            }
    985985        } else if (this.referrers instanceof OsmPrimitive[]) {
    … …  
    987987            for (OsmPrimitive ref: refs) {
    988988                if (ref.dataSet == dataSet) {
    989                     ref.visit(visitor);
     989                    ref.accept(visitor);
    990990                }
    991991            }
    … …  
    10271027     * @param visitor The visitor from which the visit() function must be called.
    10281028     */
    1029     abstract public void visit(Visitor visitor);
     1029    abstract public void accept(Visitor visitor);
    10301030
    10311031    /**

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

    r5170 r6009  
    6969                firstIteration = false;
    7070                for (RelationMember m : r.getMembers()) {
    71                     m.getMember().visit(this);
     71                    m.getMember().accept(this);
    7272                }
    7373            }
    … …  
    7676                for (OsmPrimitive osm : primitives) {
    7777                    firstIteration = true;
    78                     osm.visit(this);
     78                    osm.accept(this);
    7979                }
    8080            }

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

    r5676 r6009  
    164164    }
    165165
    166     @Override public void visit(Visitor visitor) {
     166    @Override public void accept(Visitor visitor) {
    167167        visitor.visit(this);
    168168    }
    169169
    170     @Override public void visit(PrimitiveVisitor visitor) {
     170    @Override public void accept(PrimitiveVisitor visitor) {
    171171        visitor.visit(this);
    172172    }

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

    r5170 r6009  
    6464
    6565    @Override
    66     public void visit(PrimitiveVisitor visitor) {
     66    public void accept(PrimitiveVisitor visitor) {
    6767        visitor.visit(this);
    6868    }

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

    r5847 r6009  
    209209    }
    210210
    211     @Override public void visit(Visitor visitor) {
     211    @Override public void accept(Visitor visitor) {
    212212        visitor.visit(this);
    213213    }
    214214
    215     @Override public void visit(PrimitiveVisitor visitor) {
     215    @Override public void accept(PrimitiveVisitor visitor) {
    216216        visitor.visit(this);
    217217    }

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

    r5170 r6009  
    6060
    6161    @Override
    62     public void visit(PrimitiveVisitor visitor) {
     62    public void accept(PrimitiveVisitor visitor) {
    6363        visitor.visit(this);
    6464    }

  • trunk/src/org/openstreetmap/josm/data/validation/PaintVisitor.java

    r5909 r6009  
    122122    public void visit(OsmPrimitive p) {
    123123        if (p.isUsable()) {
    124             p.visit(this);
     124            p.accept(this);
    125125        }
    126126    }

  • trunk/src/org/openstreetmap/josm/data/validation/Test.java

    r5293 r6009  
    142142        for (OsmPrimitive p : selection) {
    143143            if (p.isUsable()) {
    144                 p.visit(this);
     144                p.accept(this);
    145145            }
    146146            progressMonitor.worked(1);

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

    r5958 r6009  
    256256                    try {
    257257                        for (RelationMember em : e.getMembers()) {
    258                             em.getMember().visit(this);
     258                            em.getMember().accept(this);
    259259                        }
    260260                    } finally {
    … …  
    268268                continue;
    269269            }
    270             conflicts.getConflictForMy((OsmPrimitive)o).getTheir().visit(conflictPainter);
     270            conflicts.getConflictForMy((OsmPrimitive)o).getTheir().accept(conflictPainter);
    271271        }
    272272    }

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

    r5971 r6009  
    520520        public void visit(OsmPrimitive p) {
    521521            if (p.isUsable()) {
    522                 p.visit(this);
     522                p.accept(this);
    523523            }
    524524        }

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

    r5926 r6009  
    806806
    807807        public HistoryOsmPrimitive build(OsmPrimitive primitive) {
    808             primitive.visit(this);
     808            primitive.accept(this);
    809809            return clone;
    810810        }

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

    r5886 r6009  
    455455        final DataCountVisitor counter = new DataCountVisitor();
    456456        for (final OsmPrimitive osm : data.allPrimitives()) {
    457             osm.visit(counter);
     457            osm.accept(counter);
    458458        }
    459459        final JPanel p = new JPanel(new GridBagLayout());

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

    r5528 r6009  
    9999        { // append primitive specific
    100100            out.append("\t\"geometry\": {");
    101             p.visit(this);
     101            p.accept(this);
    102102            out.append("}");
    103103        }

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

    r5881 r6009  
    288288        osmWriter.setChangeset(changeset);
    289289        osmWriter.header();
    290         o.visit(osmWriter);
     290        o.accept(osmWriter);
    291291        osmWriter.footer();
    292292        osmWriter.flush();

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

    r5589 r6009  
    4343            switchMode("delete");
    4444            osmwriter.setWithBody(false);
    45             p.visit(osmwriter);
     45            p.accept(osmwriter);
    4646        } else {
    4747            switchMode(p.isNew() ? "create" : "modify");
    4848            osmwriter.setWithBody(true);
    49             p.visit(osmwriter);
     49            p.accept(osmwriter);
    5050        }
    5151    }
Note: See TracChangeset for help on using the changeset viewer.