Index: /applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/OSMValidatorPlugin.java
===================================================================
--- /applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/OSMValidatorPlugin.java	(revision 8904)
+++ /applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/OSMValidatorPlugin.java	(revision 8905)
@@ -38,24 +38,25 @@
     /** The list of errors per layer*/
     Map<Layer, List<TestError>> layerErrors = new HashMap<Layer, List<TestError>>();
-    
-    /** 
-     * All available tests 
-     * TODO: is there any way to find out automagically all available tests? 
-     */
-    public static Class[] allAvailableTests = new Class[]
-    { 
-    	DuplicateNode.class, 
-    	OverlappingWays.class, 
-        UntaggedNode.class, 
-        UntaggedWay.class,
-        SelfIntersectingWay.class,
-    	SpellCheck.class,
-        DuplicatedWayNodes.class, 
-        CrossingWays.class,
-        SimilarNamedWays.class,
+
+	/**
+	 * All available tests
+	 * TODO: is there any way to find out automagically all available tests?
+	 */
+	public static Class[] allAvailableTests = new Class[]
+	{
+		DuplicateNode.class,
+		OverlappingWays.class,
+		UntaggedNode.class,
+		UntaggedWay.class,
+		SelfIntersectingWay.class,
+		SpellCheck.class,
+		DuplicatedWayNodes.class,
+		CrossingWays.class,
+		SimilarNamedWays.class,
 		NodesWithSameName.class,
-        Coastlines.class,
-        WronglyOrderedWays.class,
-    };
+		Coastlines.class,
+		WronglyOrderedWays.class,
+		UnclosedWays.class,
+	};
 
 	/**
@@ -65,13 +66,13 @@
 	{
 		PreferenceEditor.importOldPreferences();
-        initializeTests( getTests() );
-	}
-	
-    @Override
+		initializeTests( getTests() );
+	}
+
+	@Override
 	public PreferenceSetting getPreferenceSetting() 
 	{
 		return new PreferenceEditor(this);
 	}
-	
+
 	@Override
 	public void mapFrameInitialized(MapFrame oldFrame, MapFrame newFrame) 
Index: /applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/tests/UnclosedWays.java
===================================================================
--- /applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/tests/UnclosedWays.java	(revision 8905)
+++ /applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/tests/UnclosedWays.java	(revision 8905)
@@ -0,0 +1,97 @@
+package org.openstreetmap.josm.plugins.validator.tests;
+
+import static org.openstreetmap.josm.tools.I18n.tr;
+
+import java.util.*;
+
+import org.openstreetmap.josm.data.osm.OsmPrimitive;
+import org.openstreetmap.josm.data.osm.Way;
+import org.openstreetmap.josm.data.coor.LatLon;
+import org.openstreetmap.josm.plugins.validator.Severity;
+import org.openstreetmap.josm.plugins.validator.Test;
+import org.openstreetmap.josm.plugins.validator.TestError;
+import org.openstreetmap.josm.plugins.validator.util.Bag;
+
+/**
+ * Check area type ways for errors
+ *
+ * @author stoecker
+ */
+public class UnclosedWays extends Test  {
+	/** The already detected errors */
+	Bag<Way, Way> _errorWays;
+
+	/**
+	 * Constructor
+	 */
+	public UnclosedWays()
+	{
+		super(tr("Unclosed Ways."),
+			  tr("This test if ways which should be areas are closed."));
+	}
+
+	@Override
+	public void startTest()
+	{
+		_errorWays = new Bag<Way, Way>();
+	}
+
+	@Override
+	public void endTest()
+	{
+		_errorWays = null;
+	}
+	
+	@Override
+	public void visit(Way w)
+	{
+		boolean force = false; /* force even if end-to-end distance is long */
+		String type = null, test;
+		
+		if( w.deleted || w.incomplete )
+			return;
+		
+		test = w.get("natural");
+		if(test != null)
+		{
+			type = tr("natural type {0}", tr(test));
+		}
+		test = w.get("landuse");
+		if(test != null)
+		{
+			type = tr("landuse type {0}", tr(test));
+		}
+		test = w.get("junction");
+		if(test != null && test.equals("roundabout"))
+		{
+			force = true;
+			type = tr("junction type {0}", tr(test));
+		}
+		test = w.get("building");
+		if (test != null && ("true".equalsIgnoreCase(test) || "yes".equalsIgnoreCase(test) || "1".equals(test)))
+		{
+			force = true;
+			type = tr("building");
+		}
+		test = w.get("area");
+		if (test != null && ("true".equalsIgnoreCase(test) || "yes".equalsIgnoreCase(test) || "1".equals(test)))
+		{
+			force = true;
+			type = tr("area");
+		}
+
+		if(type != null)
+		{
+			LatLon s = w.nodes.get(0).coor;
+			LatLon e = w.nodes.get(w.nodes.size()-1).coor;
+			/* take unclosed ways with end-to-end-distance below 10 km */
+			if(s != e && (force || s.greatCircleDistance(e) < 10000))
+			{
+				List<OsmPrimitive> primitives = new ArrayList<OsmPrimitive>();
+				primitives.add(w);
+				errors.add(new TestError(this, Severity.WARNING, tr("Unclosed way: {0}",type), primitives));
+				_errorWays.add(w,w);
+			}
+		}
+	}
+}