There have been different opinions about this patch.
There is one controversal point: Should mkgmap allow nested inner polygons?
From my point of view: YES. Because it does not harm in any way. The nested inner polygons I analysed were all reasonable. Ok, they are not 100% compliant to the specification but this is common to lots of others things in (anarchic) OSM.
At the moment I see two possible changes: 1. Issue a warning for any nested inner polygon but process it anyhow. 2. Reject nested inner polygons (and issue a warning). What a pity but they are not 100% compliant.
Please let me know what's the opinion of mkgmap community and committers because I want the improvements of error messages to be committed.
WanMil _______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
If it is not difficult to implement (also for others) then I would propose integration. If however we knew that mapnik could not implement it without major rewrites, then I would not push it forward as long as it is not at least accepted tagging for a larger minority.