<table><tr><td style="">vkrause added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15173">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D15173#318027" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D15173#318027</a>, <a href="https://phabricator.kde.org/p/pino/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@pino</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D15173#318022" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D15173#318022</a>, <a href="https://phabricator.kde.org/p/vkrause/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@vkrause</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>We could make iso-codes a hard build-time requirement and generate this file every time, to address this concern. The CI has it on all platforms but Windows already it seems. Would that be acceptable?</p></div>
</blockquote>
<p>Considering what I said earlier, it should be easy to imagine what my reply is ("no").</p></div>
</blockquote>
<p>The price for runtime loading is:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">~36kb of JSON we need to (partly) keep in non-shared heap memory vs. ~2kb of shared read-only data</li>
<li class="remarkup-list-item">some one time parsing cost which is probably irrelevant on desktop, and probably cost us ~100ms on mobile</li>
<li class="remarkup-list-item">addtional deployment of the iso-codes data file, again irrelevant on Linux desktops, probably also on Windows, but needing extra care on Android for example (or qrc bundling). Failures in deployment result in no country names being displayed at all.</li>
</ul>
<p>The price for compiled-in data is (assuming we change this to be updated on each build):</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">we might be out of sync with translations for country name changes within a release cycle, country names would be shown in the old form and without translations in that case.</li>
</ul>
<p>Obviously we disagree on how to value those trade-offs, so how to we get to a decision here?</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Also, <a href="https://phabricator.kde.org/D14163" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D14163: Fix translation of country names</a> was committed even if I had concerns on it; hence, I'm not sure it is even worth to argue here...</p></blockquote>
<p>For the record, <a href="https://phabricator.kde.org/D14163" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D14163</a> was committed after 4+ weeks of inactivity from you in the discussion, with 2 people (besides me) speaking out in favor of the patch and it blocking <a href="https://phabricator.kde.org/D14563" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D14563</a> (which addressed the other direction of this, country to iso). Additionally this patch here is attempting to address the remaining issues you pointed out in <a href="https://phabricator.kde.org/D14163" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D14163</a> by moving to the iso-codes dataset (even if we still disagree on how well it does that).</p>
<p>In all of this, let's also not forget the 18.08 baseline, where we have no working country name translations and are ~15 years out of touch with reality, all of which at a user-noticeable performance cost. I think all these patches improved that situation considerably, but there might of course still be some room for perfection ;-)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R174 PIM: KContacts</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15173">https://phabricator.kde.org/D15173</a></div></div><br /><div><strong>To: </strong>vkrause, mlaurent<br /><strong>Cc: </strong>pino, mlaurent, svuorela, kde-pim, dvasin, rodsevich, winterz, vkrause, knauss, dvratil<br /></div>