Index: trunk/src/org/openstreetmap/josm/data/osm/visitor/AllNodesVisitor.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/osm/visitor/AllNodesVisitor.java	(revision 6009)
+++ trunk/src/org/openstreetmap/josm/data/osm/visitor/AllNodesVisitor.java	(revision 6010)
@@ -48,11 +48,14 @@
             if (m.isNode()) visit(m.getNode());
     }
+    
     /**
-     * @return All nodes the given primitive has.
+     * Replies all nodes contained by the given primitives
+     * @param osms The OSM primitives to inspect
+     * @return All nodes the given primitives have.
      */
     public static Collection<Node> getAllNodes(Collection<? extends OsmPrimitive> osms) {
         AllNodesVisitor v = new AllNodesVisitor();
         for (OsmPrimitive osm : osms)
-            osm.visit(v);
+            osm.accept(v);
         return v.nodes;
     }
Index: trunk/src/org/openstreetmap/josm/data/osm/visitor/BoundingXYVisitor.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/osm/visitor/BoundingXYVisitor.java	(revision 6009)
+++ trunk/src/org/openstreetmap/josm/data/osm/visitor/BoundingXYVisitor.java	(revision 6010)
@@ -40,5 +40,5 @@
         for (RelationMember m : e.getMembers()) {
             if (!m.isRelation()) {
-                m.getMember().visit(this);
+                m.getMember().accept(this);
             }
         }
@@ -130,5 +130,5 @@
                 continue;
             }
-            p.visit(this);
+            p.accept(this);
         }
     }
Index: trunk/src/org/openstreetmap/josm/data/osm/visitor/MergeSourceBuildingVisitor.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/osm/visitor/MergeSourceBuildingVisitor.java	(revision 6009)
+++ trunk/src/org/openstreetmap/josm/data/osm/visitor/MergeSourceBuildingVisitor.java	(revision 6010)
@@ -134,5 +134,5 @@
         //
         for (Node n: w.getNodes()) {
-            n.visit(this);
+            n.accept(this);
         }
         // ... and the way itself
@@ -152,5 +152,5 @@
             }
             if (isInSelectionBase(member.getMember()) || member.getMember().isNew()) {
-                member.getMember().visit(this);
+                member.getMember().accept(this);
             } else {
                 rememberIncomplete(member.getMember());
@@ -185,5 +185,5 @@
     public DataSet build() {
         for (OsmPrimitive primitive: selectionBase.getAllSelected()) {
-            primitive.visit(this);
+            primitive.accept(this);
         }
         buildHull();
Index: trunk/src/org/openstreetmap/josm/data/osm/visitor/PrimitiveVisitor.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/osm/visitor/PrimitiveVisitor.java	(revision 6009)
+++ trunk/src/org/openstreetmap/josm/data/osm/visitor/PrimitiveVisitor.java	(revision 6010)
@@ -6,7 +6,26 @@
 import org.openstreetmap.josm.data.osm.IWay;
 
+/**
+ * OSM primitives interfaces visitor, following conventional <a href="http://en.wikipedia.org/wiki/Visitor_pattern">visitor design pattern</a>.
+ * @since 4100
+ */
 public interface PrimitiveVisitor {
+    
+    /**
+     * Visiting call for points.
+     * @param n The node to inspect.
+     */
     void visit(INode n);
+    
+    /**
+     * Visiting call for lines.
+     * @param w The way to inspect.
+     */
     void visit(IWay w);
-    void visit(IRelation e);
+
+    /**
+     * Visiting call for relations.
+     * @param r The relation to inspect.
+     */
+    void visit(IRelation r);
 }
Index: trunk/src/org/openstreetmap/josm/data/osm/visitor/Visitor.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/osm/visitor/Visitor.java	(revision 6009)
+++ trunk/src/org/openstreetmap/josm/data/osm/visitor/Visitor.java	(revision 6010)
@@ -10,4 +10,5 @@
  * Implementation of the visitor scheme. Every @{link org.openstreetmap.josm.data.OsmPrimitive}
  * can be visited by several different visitors.
+ * @since 8
  */
 public interface Visitor {
@@ -20,14 +21,17 @@
      * Visiting call for lines.
      * @param w The way to inspect.
+     * @since 64
      */
     void visit(Way w);
     /**
      * Visiting call for relations.
-     * @param e The relation to inspect.
+     * @param r The relation to inspect.
+     * @since 343
      */
-    void visit(Relation e);
+    void visit(Relation r);
     /**
      * Visiting call for changesets.
      * @param cs The changeset to inspect.
+     * @since 1523
      */
     void visit(Changeset cs);
Index: trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/WireframeMapRenderer.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/WireframeMapRenderer.java	(revision 6009)
+++ trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/WireframeMapRenderer.java	(revision 6010)
@@ -161,5 +161,5 @@
         for (final Relation rel : data.searchRelations(bbox)) {
             if (rel.isDrawable() && !ds.isSelected(rel) && !rel.isDisabledAndHidden()) {
-                rel.visit(this);
+                rel.accept(this);
             }
         }
@@ -176,5 +176,5 @@
                     untaggedWays.add(way);
                 } else {
-                    way.visit(this);
+                    way.accept(this);
                 }
             }
@@ -185,5 +185,5 @@
         for (List<Way> specialWays : Arrays.asList(new List[]{untaggedWays, highlightedWays})) {
             for (final Way way : specialWays){
-                way.visit(this);
+                way.accept(this);
             }
             specialWays.clear();
@@ -193,5 +193,5 @@
         for (final OsmPrimitive osm : data.getSelected()) {
             if (osm.isDrawable()) {
-                osm.visit(this);
+                osm.accept(this);
             }
         }
@@ -201,5 +201,5 @@
             if (osm.isDrawable() && !ds.isSelected(osm) && !osm.isDisabledAndHidden())
             {
-                osm.visit(this);
+                osm.accept(this);
             }
         }
Index: trunk/src/org/openstreetmap/josm/data/validation/tests/UntaggedNode.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/validation/tests/UntaggedNode.java	(revision 6009)
+++ trunk/src/org/openstreetmap/josm/data/validation/tests/UntaggedNode.java	(revision 6010)
@@ -48,5 +48,5 @@
         for (OsmPrimitive p : selection) {
             if (p.isUsable() && p instanceof Node) {
-                p.visit(this);
+                p.accept(this);
             }
         }
Index: trunk/src/org/openstreetmap/josm/data/validation/util/AggregatePrimitivesVisitor.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/validation/util/AggregatePrimitivesVisitor.java	(revision 6009)
+++ trunk/src/org/openstreetmap/josm/data/validation/util/AggregatePrimitivesVisitor.java	(revision 6010)
@@ -30,5 +30,5 @@
     public Collection<OsmPrimitive> visit(Collection<OsmPrimitive> data) {
         for (OsmPrimitive osm : data) {
-            osm.visit(this);
+            osm.accept(this);
         }
 
@@ -58,5 +58,5 @@
             aggregatedData.add(r);
             for (RelationMember m : r.getMembers()) {
-                m.getMember().visit(this);
+                m.getMember().accept(this);
             }
         }
Index: trunk/src/org/openstreetmap/josm/data/validation/util/MultipleNameVisitor.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/validation/util/MultipleNameVisitor.java	(revision 6009)
+++ trunk/src/org/openstreetmap/josm/data/validation/util/MultipleNameVisitor.java	(revision 6010)
@@ -50,5 +50,5 @@
             }
 
-            osm.visit(this);
+            osm.accept(this);
             if (multipleClassname == null) {
                 multipleClassname = className;