[mkgmap-dev] [mkgmap-svn] Commit r4810: revert changes from r4809 for now, they caused more trouble

Mon Nov 1 11:08:02 GMT 2021

Hi

I've just tried this and leaving in the original calcMdr20SortPosn and
checking its value and also doing your version and checking and they
all come up with the same answer.

On the full NRW map from Arndt (253 tiles) the extra sort adds 21
seconds to 4 mins 21 secs do generate gmapsupp from the tile .imgs

Just tried without the original calcMdr20SortPosn and back to the
original time - the 2 version with the sort take the same time 

Ticker

On Mon, 2021-11-01 at 10:32 +0000, Gerd Petermann wrote:
> Hi Ticker,
> 
> I think I tried that first and got an assertion or a different
> result. Maybe I didn't do it right?
> 
> Gerd
> 
> ________________________________________
> Von: mkgmap-dev <mkgmap-dev-bounces at lists.mkgmap.org.uk> im Auftrag
> von Ticker Berkin <rwb-mkgmap at jagit.co.uk>
> Gesendet: Montag, 1. November 2021 11:29
> An: Development list for mkgmap
> Betreff: Re: [mkgmap-dev] [mkgmap-svn] Commit r4810: revert changes
> from r4809 for now, they caused more trouble
> 
> Hi Gerd
> 
> Looking at what Mdr5::calcMdr20SortPos() is doing, can't it be
> simplified away to just an extra call in genCitiesAndMdr20s():
> 
>                         ... around line 106
>                         if (!c.isSameByName(collator, lastCity))
>                                 mdr20count++;
>                         c.setMdr20Index(mdr20count);
> add this:               c.setMdr20SortPos(mdr20count);
> 
> Ticker
> 
> 
> On Sat, 2021-10-30 at 09:16 +0000, Gerd Petermann wrote:
> > Hi Ticker,
> > 
> > attached is a patch that simplifies calcMdr20SortPos(), no change
> > in
> > output intended.
> > Does it help?
> > 
> > Gerd
> 
> 
> _______________________________________________
> mkgmap-dev mailing list
> mkgmap-dev at lists.mkgmap.org.uk
> https://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
> _______________________________________________
> mkgmap-dev mailing list
> mkgmap-dev at lists.mkgmap.org.uk
> https://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev