D14563: Add CLDR-based country to ISO code mapping
Volker Krause
noreply at phabricator.kde.org
Thu Aug 2 18:07:33 BST 2018
vkrause created this revision.
Restricted Application added a project: KDE PIM.
Restricted Application added a subscriber: kde-pim.
vkrause requested review of this revision.
REVISION SUMMARY
This replaces the many year outdated countrytransl.map file with a
compiled-in data table. This is much more complete (28k strings rather
than the previous ~5k), up-to-date (the old data set was from a time
when Yugoslavia was still a thing), and much much faster to access
(no more text file parsing for each lookup, but a binary search in
shared read-only data). However, we pay for this by needing more disk
space (670kB vs 95kB), which is largely due to the increased amount of
strings, many of which are in "expensive" encodings, the overhead per
string only increases from four to five bytes.
This is based on Sune's work in kde:scratch/sune/countrieslistcldr.
REPOSITORY
R174 PIM: KContacts
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D14563
AFFECTED FILES
autotests/addresstest.cpp
src/CMakeLists.txt
src/address.cpp
src/countrytoisomap_data.cpp
src/countrytoisomap_p.h
src/generator/CMakeLists.txt
src/generator/main.cpp
src/generator/translatedcountrylist.cpp
src/generator/translatedcountrylist.h
To: vkrause
Cc: kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20180802/9da6becf/attachment.html>
More information about the kde-pim
mailing list