[mkgmap-dev] Index: MDR12 breaking POI search

Wed Jan 19 21:42:28 GMT 2011

Am 19.01.2011 22:07, schrieb Johann Gail:
>
>
> WanMil schrieb:
>> Am 19.01.2011 20:15, schrieb WanMil:
>>
>>> I started to strip down the index branch to things that work so that we
>>> can add more and more things until we get a running index branch.
>>>
>>> As a first step I modified the OSM reader so that there will be no
>>> special character for sure. All non recognized characters
>>> ('a'-'z','A'-'Z','0'-'9',' ','-') are replaced by 'Z'. As far as I
>>> understood the character encoding might be a problem. So replacing all
>>> problematic characters might help. (see attached patch)
>>>
>>> 2nd: I splitted Germany with the default splitter settings so that I get
>>> big enough tiles.
>>>
>>> 3rd: I removed MDR8 and MDR12 from the MDRFile class.
>>>
>>> 4th: I compiled a map using 2 tiles from my Germany split.
>>>
>>> The result was interesting:
>>> * POI search works
>>> * All region search fields are disabled
>>> * I don't see any more exceptions or crashes in MapSource
>>> * Street search still has the problem with the message "Selected street
>>> is not allowed in this map product" (translated from german...)
>>> * Most of the country information is set to ABC. I don't know if that
>>> has something to do with removing the --make-poi-index option?
>>>
>>>
>>> I observed the same result when I added the MDR8.
>>> Adding MDR12 returns to the old behaviour with exceptions and MapSource
>>> crashes.
>>>
>>> WanMil
>>>
>>>
>>
>> Compiling a map
>> - with the index_nospecialchars_v1.patch: Region field disabled
>> - without the index_nospecialchars_v1.patch: Region field enabled
>>
>> I do not understand...
>> WanMil
>>
>>
> Just found at
> http://pkmaps.freeforums.org/to-enable-find-search-address-option-for-maps-in-gpsmapedit-t61.html
>
>
> "Note: Always use UPPER CASE for defining Country, Region, City etc
> because many Garmin devices can only allow users to text in alphabets in
> Upper Case."
>
> Maybe you should modify your patch to only allow upper case letters.
>
>
> Regards,
> Johann

That might be a good hint but I think Steve already does this in the MDR 
creation process?!

The patch must allow lower case letters. Otherwise all highway=primary 
will become highway=PRIMARY and all styles are no longer working...

WanMil