Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/DuplicateNode.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/DuplicateNode.java	(revision 6240)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/DuplicateNode.java	(revision 6241)
@@ -153,4 +153,10 @@
     }
 
+    /**
+     * Returns the list of "duplicate nodes" errors for the given selection of node and parent test
+     * @param parentTest The parent test of returned errors
+     * @param nodes The nodes selction to look into
+     * @return the list of "duplicate nodes" errors
+     */
     public List<TestError> buildTestErrors(Test parentTest, List<Node> nodes) {
         List<TestError> errors = new ArrayList<TestError>();
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/DuplicatedWayNodes.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/DuplicatedWayNodes.java	(revision 6240)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/DuplicatedWayNodes.java	(revision 6241)
@@ -15,7 +15,14 @@
 import org.openstreetmap.josm.data.validation.TestError;
 
+/**
+ * Checks for ways with identical consecutive nodes.
+ * @since 3669
+ */
 public class DuplicatedWayNodes extends Test {
     protected static final int DUPLICATE_WAY_NODE = 501;
 
+    /**
+     * Constructs a new {@code DuplicatedWayNodes} test.
+     */
     public DuplicatedWayNodes() {
         super(tr("Duplicated way nodes"),
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/NodesDuplicatingWayTags.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/NodesDuplicatingWayTags.java	(revision 6240)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/NodesDuplicatingWayTags.java	(revision 6241)
@@ -27,4 +27,7 @@
     protected static final int NODE_DUPING_PARENT_WAY_TAGS = 2401;
 
+    /**
+     * Constructs a new {@code NodesDuplicatingWayTags} test.
+     */
     public NodesDuplicatingWayTags() {
         super(tr("Nodes duplicating way tags"),
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/NodesWithSameName.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/NodesWithSameName.java	(revision 6240)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/NodesWithSameName.java	(revision 6241)
@@ -16,4 +16,8 @@
 import org.openstreetmap.josm.gui.progress.ProgressMonitor;
 
+/**
+ * Finds nodes that have the same name (might be duplicates)
+ * @since 3669
+ */
 public class NodesWithSameName extends Test {
     protected static final int SAME_NAME = 801;
@@ -21,4 +25,7 @@
     private Map<String, List<Node>> namesToNodes;
 
+    /**
+     * Constructs a new {@code NodesWithSameName} test.
+     */
     public NodesWithSameName() {
         super(tr("Nodes with same name"),
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/OverlappingAreas.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/OverlappingAreas.java	(revision 6240)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/OverlappingAreas.java	(revision 6241)
@@ -18,4 +18,8 @@
 import org.openstreetmap.josm.tools.Utils;
 
+/**
+ * Checks if areas overlap.
+ * @since 4448
+ */
 public class OverlappingAreas extends Test {
 
@@ -23,4 +27,7 @@
     protected QuadBuckets<Way> index = new QuadBuckets<Way>();
 
+    /**
+     * Constructs a new {@code OverlappingAreas} test.
+     */
     public OverlappingAreas() {
         super(tr("Overlapping Areas"), tr("This test checks if areas overlap."));
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/PowerLines.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/PowerLines.java	(revision 6240)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/PowerLines.java	(revision 6241)
@@ -34,7 +34,11 @@
     protected static final int POWER_LINES = 2501;
     
+    /** Values for {@code power} key interpreted as power lines */
     public static final Collection<String> POWER_LINE_TAGS = Arrays.asList("line", "minor_line");
+    /** Values for {@code power} key interpreted as power towers */
     public static final Collection<String> POWER_TOWER_TAGS = Arrays.asList("tower", "pole");
+    /** Values for {@code power} key interpreted as power stations */
     public static final Collection<String> POWER_STATION_TAGS = Arrays.asList("station", "sub_station", "plant", "generator");
+    /** Values for {@code power} key interpreted as allowed power items */
     public static final Collection<String> POWER_ALLOWED_TAGS = Arrays.asList("switch", "transformer", "busbar", "generator");
     
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java	(revision 6240)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java	(revision 6241)
@@ -83,5 +83,4 @@
         private Collection<Node> nodes = new LinkedList<Node>();
         private Collection<Way> ways = new LinkedList<Way>();
-        private Collection<Way> closedways = new LinkedList<Way>();
         private Collection<Way> openways = new LinkedList<Way>();
         private Collection<Relation> relations = new LinkedList<Relation>();
@@ -129,7 +128,5 @@
                 } else if(m.isWay()) {
                     ri.ways.add(m.getWay());
-                    if (m.getWay().isClosed()) {
-                        ri.closedways.add(m.getWay());
-                    } else {
+                    if (!m.getWay().isClosed()) {
                         ri.openways.add(m.getWay());
                     }
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/SelfIntersectingWay.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/SelfIntersectingWay.java	(revision 6240)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/SelfIntersectingWay.java	(revision 6241)
@@ -20,4 +20,7 @@
     protected static final int SELF_INTERSECT = 401;
 
+    /**
+     * Constructs a new {@code SelfIntersectingWay} test.
+     */
     public SelfIntersectingWay() {
         super(tr("Self-intersecting ways"),
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/TurnrestrictionTest.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/TurnrestrictionTest.java	(revision 6240)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/TurnrestrictionTest.java	(revision 6241)
@@ -18,4 +18,8 @@
 import org.openstreetmap.josm.data.validation.TestError;
 
+/**
+ * Checks if turnrestrictions are valid
+ * @since 3669
+ */
 public class TurnrestrictionTest extends Test {
 
@@ -36,4 +40,7 @@
     protected static final int SUPERFLUOUS = 1815;
 
+    /**
+     * Constructs a new {@code TurnrestrictionTest}.
+     */
     public TurnrestrictionTest() {
         super(tr("Turnrestrictions"), tr("This test checks if turnrestrictions are valid"));
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java	(revision 6240)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java	(revision 6241)
@@ -21,5 +21,4 @@
 import org.openstreetmap.josm.data.coor.LatLon;
 import org.openstreetmap.josm.data.osm.BBox;
-import org.openstreetmap.josm.data.osm.DataSet;
 import org.openstreetmap.josm.data.osm.Node;
 import org.openstreetmap.josm.data.osm.OsmUtils;
@@ -48,5 +47,4 @@
     private Set<Node> othernodes; // nodes appearing at least twice
     private Area dsArea;
-    private DataSet ds;
 
     private double mindist;
@@ -71,6 +69,5 @@
         mindist = Main.pref.getDouble(PREFIX + ".node_way_distance", 10.0);
         minmiddledist = Main.pref.getDouble(PREFIX + ".way_way_distance", 0.0);
-        this.ds = Main.main.getCurrentDataSet();
-        this.dsArea = ds.getDataSourceArea();
+        this.dsArea = Main.main.getCurrentDataSet().getDataSourceArea();
     }
 
Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/WayConnectedToArea.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/validation/tests/WayConnectedToArea.java	(revision 6240)
+++ /trunk/src/org/openstreetmap/josm/data/validation/tests/WayConnectedToArea.java	(revision 6241)
@@ -16,6 +16,13 @@
 import org.openstreetmap.josm.gui.mappaint.ElemStyles;
 
+/**
+ * Checks for ways connected to areas.
+ * @since 4682
+ */
 public class WayConnectedToArea extends Test {
 
+    /**
+     * Constructs a new {@code WayConnectedToArea} test.
+     */
     public WayConnectedToArea() {
         super(tr("Way connected to Area"), tr("Checks for ways connected to areas."));