[mkgmap-dev] [PATCH v2] MP: Close ways with both endpoints outside the bounding box

Thu Mar 4 21:12:57 GMT 2010

> Hi WanMil,
>
>> The patch closes all ways having both endpoints on the same side
>> outside the bounding box. In this case it is not necessary to check
>> if closing the way intersects the way itself.
>>
>> This fixes most (but not all) of the outstanding tile splitter
>> problems.
>
> I still see some warnings presumably due to tile splitter, but other
> than that, this and mp_lesscuts_v2.patch look fine to me. Compared to
> the unpatched mkgmap r1593, these two patches shaved more than 11k off
> the warning log (181452 bytes for today, 192079 for yesterday, even
> though today's run contained many non-multipolygon warnings).
>
> 	Marko

Torsten Leistikow pointed me to another problem with the tile bounds. In 
case a polygon was splitted by the bounding box into different distinct 
polygons only the first of it was taken.

Version 2 of this patch handles this case.

Summary for version 2:
- Closing ways with both endpoints outside the bounding box and with 
both orientation towards the bounding box.
- Bugfix: if a polygon was splitted by the bounding box into different 
distinct polygons only the first of it was taken. Now all distinct 
polygons are processed. (thanks to Torsten Leistikow)

I give a very big "GO for commit" to this patch :-)

WanMil
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: mp_closeways_outbbox_v2.patch
Url: http://lists.mkgmap.org.uk/pipermail/mkgmap-dev/attachments/20100304/b481647b/attachment.pl 