Re: [mkgmap-dev] Commit: r1050: Improve performance of distance calculations.

2009-05-30 Thread Mark Burton
Hi Johann, > i didn't want to upset you. Actually, you didn't - I was just cruelly making fun at your expense. Sorry, I can't help it sometimes. I really don't care much what the functions are called, especially as slowDistance() is no longer used anyway (apart from the check in quickDistance()

Re: [mkgmap-dev] Commit: r1050: Improve performance of distance calculations.

2009-05-30 Thread Johann Gail
Hi Mark, i didn't want to upset you. Its your code, well code btw., and you can name the functions as you like. But from the sight of a progrommer not so deep in the internals of mkgmap, it seems to me clearer to choose from the functions quickDistance() and exactDistance(). May I propose

Re: [mkgmap-dev] Commit: r1050: Improve performance of distance calculations.

2009-05-30 Thread Mark Burton
Hi Johann, > May I propose to rename the function slowDistance() to exactDistance()? > I think the main reason for this function is not to be slow, but to be > the exact calculation. It doesn't want to be slow but it just is because acos() is a complete dog. But how do you know it's exact anyw

Re: [mkgmap-dev] Build an Italian OSM Map on Windows

2009-05-30 Thread Carlos Dávila
Marco Certelli escribió: > Hi all. For people interested in building an Italian map (but actually any > country is supported) on a Windows Machine (maybe I'm the only one...), well > I prepared a script that automates the process of > downloading/splitting/building the map. MapSource Install is

Re: [mkgmap-dev] Commit: r1050: Improve performance of distance calculations.

2009-05-30 Thread Johann Gail
May I propose to rename the function slowDistance() to exactDistance()? I think the main reason for this function is not to be slow, but to be the exact calculation. Regards, Johann Original distance() now renamed to slowDistance() and distance() now calls quickDistance(). __

Re: [mkgmap-dev] Commit: r1051: Loop splitting code now copes with ways that contain nested loops.

2009-05-30 Thread Mark Burton
Hi Felix, > Working great. Autria.osm.bz2 from Geofabrik compiles without any > unroutable roads now. I checked some that with 1049 still would not > route and they do route. Very good. > Quick Distance also working without problems - though speed improvement > only up to 40% (but hey that's

Re: [mkgmap-dev] Commit: r1051: Loop splitting code now copes with ways that contain nested loops.

2009-05-30 Thread Felix Hartmann
Working great. Autria.osm.bz2 from Geofabrik compiles without any unroutable roads now. I checked some that with 1049 still would not route and they do route. Quick Distance also working without problems - though speed improvement only up to 40% (but hey that's a lot too, after your reports I th

[mkgmap-dev] Quick distance and loop split fix committed

2009-05-30 Thread Mark Burton
Hopefully, these patches have been tested enough to show up any gross problems - only received good reports so far. If anyone has any issues, let me know and I will revert/fix as appropriate. Cheers, Mark ___ mkgmap-dev mailing list mkgmap-dev@lists.m

[mkgmap-dev] Build an Italian OSM Map on Windows

2009-05-30 Thread Marco Certelli
Hi all. For people interested in building an Italian map (but actually any country is supported) on a Windows Machine (maybe I'm the only one...), well I prepared a script that automates the process of downloading/splitting/building the map. MapSource Install is also supported (.reg) I also pr

[mkgmap-dev] Commit: r1051: Loop splitting code now copes with ways that contain nested loops.

2009-05-30 Thread svn commit
Version 1051 was commited by markb on 2009-05-30 10:04:30 +0100 (Sat, 30 May 2009) Loop splitting code now copes with ways that contain nested loops. Before, having detected a loop and split the way into two, it did not then go on to check whether the first portion contained any more loops. It

[mkgmap-dev] Commit: r1050: Improve performance of distance calculations.

2009-05-30 Thread svn commit
Version 1050 was commited by markb on 2009-05-30 10:04:26 +0100 (Sat, 30 May 2009) Improve performance of distance calculations. Moved distance() from MultiPolygonRelation class into Coord class to form the basis of quickDistance(). Original distance() now renamed to slowDistance() and distanc