Review Request 118692: Fix reading of entries for language/country combinations
Matthew Dawson
matthew at mjdsystems.ca
Fri Jun 20 16:23:17 UTC 2014
> On June 17, 2014, 6:50 p.m., Matthew Dawson wrote:
> > autotests/kdesktopfiletest.cpp, line 88
> > <https://git.reviewboard.kde.org/r/118692/diff/2-3/?file=280487#file280487line88>
> >
> > Please add a test for both:
> > - when both the country/language match and a modifier is included.
> > - When the language is omitted, the country matches, and a modifier is included.
>
> Martin Gräßlin wrote:
> omitting language doesn't make sense, or does it? Given the spec this could not happen
Sorry, I mistyped that. I meant omitting the country, but language and modifier included. Sorry about that.
- Matthew
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118692/#review60336
-----------------------------------------------------------
On June 20, 2014, 4:56 a.m., Martin Gräßlin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118692/
> -----------------------------------------------------------
>
> (Updated June 20, 2014, 4:56 a.m.)
>
>
> Review request for KDE Frameworks, David Faure, John Layt, and Oswald Buddenhagen.
>
>
> Repository: kconfig
>
>
> Description
> -------
>
> 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.
>
>
> Diffs
> -----
>
> autotests/kdesktopfiletest.h f854d6cc7022a9819e36fe347bc6876991cfe8bf
> autotests/kdesktopfiletest.cpp 6c3af4c2cd55b9140c0cd48526947431ceb7e798
> src/core/kconfig.cpp b93199948360d5f4280b49461a93ef39cdf748e5
> src/core/kconfigdata.h fdec85dc90467560bd312b72266ef0b3f243d076
> src/core/kconfigdata.cpp 109063d65e97bcb7ba08cf6e5a6f3acb885fe845
> src/core/kconfigini.cpp f4ce94107ff6ca67dc1989d39ad82623bb9fd216
>
> Diff: https://git.reviewboard.kde.org/r/118692/diff/
>
>
> Testing
> -------
>
> unit tests still pass, but my knowledge about KConfig and locales is not sufficient to be sure that this is now 100 % correct.
>
>
> Thanks,
>
> Martin Gräßlin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140620/18542a5f/attachment.html>
More information about the Kde-frameworks-devel
mailing list