Re: [mkgmap-dev] [PATCH v4] Multipolygon code enabled for overlapping lines

2010-02-17 Thread WanMil
> Hi WanMil, > >> I didn't get any negative feedback. Could you please commit this >> patch? > You did get some feedback from me, but it is not that critical. I > committed the patch in trunk r1575. > Thank you for your ongoing efforts, > > Marko Yeah, but it was not too *negative* ;-) Than

Re: [mkgmap-dev] [PATCH v4] Multipolygon code enabled for overlapping lines

2010-02-17 Thread Marko Mäkelä
Hi WanMil, > I didn't get any negative feedback. Could you please commit this > patch? You did get some feedback from me, but it is not that critical. I committed the patch in trunk r1575. Thank you for your ongoing efforts, Marko ___ mkgmap

Re: [mkgmap-dev] [PATCH v4] Multipolygon code enabled for overlapping lines

2010-02-17 Thread WanMil
> Compared to v3 (posted by Carlos in thread "Wrong multipolygon > warnings") some unused debug messages have been removed. > > The patch enables the multipolygon code to process multipolygons with > overlapping lines. > > WanMil > Marko, Steve, I didn't get any negative feedback. Could you pleas

Re: [mkgmap-dev] [PATCH v4] Multipolygon code enabled for overlapping lines

2010-02-16 Thread Marko Mäkelä
Hi WanMil, > if you want to know exactly what's happening please start mkgmap with > debug-level FINE. I don't do anything else. I am happy for any good > hint what might be a problem. But I don't investigate every warning > of the mp code. Fair enough. For now, I will blame it on the split

Re: [mkgmap-dev] [PATCH v4] Multipolygon code enabled for overlapping lines

2010-02-15 Thread WanMil
> Hi WanMil, > >> I downloaded the finland OSM dump from geofabrik from today (Feb >> 15th). >> I used splitter v105 and the areas.list file from >> http://www.polkupyoraily.net/osm/ to get three tiles. >> Then I tried that patch: >> * I got no warning for mp 404644 >> * I got only one "unknown rea

Re: [mkgmap-dev] [PATCH v4] Multipolygon code enabled for overlapping lines

2010-02-15 Thread Marko Mäkelä
Hi WanMil, > I downloaded the finland OSM dump from geofabrik from today (Feb > 15th). > I used splitter v105 and the areas.list file from > http://www.polkupyoraily.net/osm/ to get three tiles. > Then I tried that patch: > * I got no warning for mp 404644 > * I got only one "unknown reason" warni

Re: [mkgmap-dev] [PATCH v4] Multipolygon code enabled for overlapping lines

2010-02-15 Thread WanMil
> Hi WanMil, > >> Compared to v3 (posted by Carlos in thread "Wrong multipolygon >> warnings") some unused debug messages have been removed. >> >> The patch enables the multipolygon code to process multipolygons with >> overlapping lines. > > For the Geofabrik Finland extract of today, the patch re

Re: [mkgmap-dev] [PATCH v4] Multipolygon code enabled for overlapping lines

2010-02-14 Thread Marko Mäkelä
Hi WanMil, > Compared to v3 (posted by Carlos in thread "Wrong multipolygon > warnings") some unused debug messages have been removed. > > The patch enables the multipolygon code to process multipolygons with > overlapping lines. For the Geofabrik Finland extract of today, the patch reduces th

[mkgmap-dev] [PATCH v4] Multipolygon code enabled for overlapping lines

2010-02-14 Thread WanMil
Compared to v3 (posted by Carlos in thread "Wrong multipolygon warnings") some unused debug messages have been removed. The patch enables the multipolygon code to process multipolygons with overlapping lines. WanMil Index: src/uk/me/parabola/mkgmap/reader/osm/MultiPolygonRelation.java ===