<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/118692/">https://git.reviewboard.kde.org/r/118692/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On June 18th, 2014, 3:04 a.m. EDT, <b>Oswald Buddenhagen</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">i don't see that you are proactively excluding the @modifier. it would seem that it's effectively part of the country, and the test works only because it's expected to fall back to de instead of de_AT anyway.
there is another consideration ... matching follows a strict hierarchy, so an entry for de_DE will never be used to satisfy a request for de. arguably, that's correct - it becomes obvious when there is both de_DE and de_AT to choose from. even less, de_DE is used to satisfy a request for de_AT, which seems even more obvious. otoh, this stuff is mostly used for translating desktop files, so one can argue that being strict puts unreasonable demands on the population of the entries. i suggest you talk to albert.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Actually, QLocale doesn't give us a way to get the @modifier information currently. John mentioned that proper support is coming eventually, but right now we have no way of extracting it. Once support is available, @modifier support should be added. Right now any @modifier is stripped away, and any translation using it will be passed over.
I'm going by the specification here: http://standards.freedesktop.org/desktop-entry-spec/latest/ar01s04.html that this is the right behaviour. I'm not sure what your second paragraph is saying, so I'm not sure how it deviates from that link. I tried finding a policy about desktop files and translation for KDE, but my Google-fu didn't find anything. I would think we should move towards following the spec above as much as possible, otherwise our menu entries will show up differently in other desktops.</pre>
<br />
<p>- Matthew</p>
<br />
<p>On June 16th, 2014, 4:26 a.m. EDT, Martin Gräßlin wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Frameworks, David Faure, John Layt, and Oswald Buddenhagen.</div>
<div>By Martin Gräßlin.</div>
<p style="color: grey;"><i>Updated June 16, 2014, 4:26 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kconfig
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Fix reading of entries for language/country combinations
This fixes a regression introduced in
988f09bb051dca0437ecec431ee44ed5b4a560d8.
The mentioned commit ensures that if the locale is e.g. "de_DE" the
entry "de" will be used. But this breaks if there is a translation
for another country. E.g. for "de_CH" it would also pick the "de"
entry.
This change now operates on both just the language code and the locale.
If an entry with the language code is present it will be picked. If
another entry with the exact locale is found it will be overwritten.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">unit tests still pass, but my knowledge about KConfig and locales is not sufficient to be sure that this is now 100 % correct.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>autotests/kdesktopfiletest.cpp <span style="color: grey">(6c3af4c2cd55b9140c0cd48526947431ceb7e798)</span></li>
<li>src/core/kconfig.cpp <span style="color: grey">(a2598f8e8fce91a8de3f34b272e15a6c280a50db)</span></li>
<li>src/core/kconfigdata.h <span style="color: grey">(fdec85dc90467560bd312b72266ef0b3f243d076)</span></li>
<li>src/core/kconfigdata.cpp <span style="color: grey">(109063d65e97bcb7ba08cf6e5a6f3acb885fe845)</span></li>
<li>src/core/kconfigini.cpp <span style="color: grey">(a882ee31150658f3e5cfb036362ff0583f71cbd9)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/118692/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>