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

Thu Nov 4 13:34:20 GMT 2021

Hi Gerd

This was a bit arbitrary and maybe I should have reverted it to
.equals().

Generally Regions and Countries don't cause a problem because they
(almost always) originate from --bounds and go through
PlacesFile.createRegion() / createCountry() which stops any case
difference within a tile.

Ticker

On Thu, 2021-11-04 at 13:15 +0000, Gerd Petermann wrote:
> Hi Ticker,
> 
> why collator.setStrength(Collator.SECONDARY); in Mdr29?
> 
> Gerd
> 
> ________________________________________
> Von: mkgmap-dev <mkgmap-dev-bounces at lists.mkgmap.org.uk> im Auftrag
> von Ticker Berkin <rwb-mkgmap at jagit.co.uk>
> Gesendet: Donnerstag, 4. November 2021 12:40
> 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
> 
> Yes, patch r4809 caused the crash with Arndt's data (--lower-case and
> differently cased city names).
> 
> Ticker
> 
> On Thu, 2021-11-04 at 11:11 +0000, Gerd Petermann wrote:
> > Hi Ticker,
> > 
> > thanks, will have a closer later. Just to make sure:
> > My understanding is that the assertion with Arndts data was caused
> > by
> > your patch (r4809) and everything worked fine with r4808 / r4810.
> > 
> > Gerd
> > 
> > ________________________________________
> > Von: mkgmap-dev <mkgmap-dev-bounces at lists.mkgmap.org.uk> im Auftrag
> > von Ticker Berkin <rwb-mkgmap at jagit.co.uk>
> > Gesendet: Donnerstag, 4. November 2021 11:55
> > 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
> > 
> > Here is a minimal patch to stop the 2 assertion crashes:
> > 1/ unspecified sort for some blocks of Unicode chararacters.
> > 2/ inconsistent MDR20 city positions that could result from city
> > names
> >    with different case or use of non-unicode multi-byte codepage
> > 
> > For 2/ with option --lower-case, the behaviour of how and which
> > cities
> > with capitalisation differences are shown in the find list, and
> > which
> > streets are attached to them is:
> > 1/ device/BaseCamp/MapSource dependant.
> > 2/ depends on how the streets are spread over tiles where there is
> >    possible name conflict.
> > 3/ Affected by the presence of city POI on a tile.
> > 
> > Ticker
> > 
> > _______________________________________________
> > 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
> _______________________________________________
> mkgmap-dev mailing list
> mkgmap-dev at lists.mkgmap.org.uk
> https://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev