Current mp implementation does not remove polygon tags from unclosed ways. I have observed that the unclosed polygons sometimes are closed in a later step of mkgmap. This causes flooding of areas around seas. Finnland is a great test area for this.

Attached patch removes all polygon tags from ways that are used in multipolygons.

WanMil
Index: src/uk/me/parabola/mkgmap/reader/osm/MultiPolygonRelation.java
===================================================================
--- src/uk/me/parabola/mkgmap/reader/osm/MultiPolygonRelation.java      
(revision 1533)
+++ src/uk/me/parabola/mkgmap/reader/osm/MultiPolygonRelation.java      
(working copy)
@@ -603,11 +603,14 @@
                                        // so we remove all tags
                                        currentRing.ring.removeAllTagsDeep();
                                } else {
-                                       // the ring has been composed by 
several ways
-                                       // they may contain line tags
-                                       // however all polygon tags are not 
processed
-                                       // because they are only lines and not 
polygons
-                                       // so we don't have to remove any tag
+                                       // remove all polygons tags from the 
original ways
+                                       // sometimes the ways seem to be 
autoclosed later on
+                                       // in mkgmap
+                                       for (Way w : 
currentRing.ring.getOriginalWays()) {
+                                               for (String polygonTag : 
polygonTags) {
+                                                       w.deleteTag(polygonTag);
+                                               }
+                                       }
                                }
 
                                boolean useRelationTags = currentRing.outer
_______________________________________________
mkgmap-dev mailing list
[email protected]
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Reply via email to