Review Request 118564: Fix locale-aware reading in KDesktopFile
Martin Gräßlin
mgraesslin at kde.org
Tue Jun 10 06:42:04 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118564/
-----------------------------------------------------------
(Updated June 10, 2014, 8:42 a.m.)
Review request for KDE Frameworks and John Layt.
Changes
-------
Storing and resetting the default using a small helper class. It's currently duplicating the code, any preferences on where to put shared code from the tests or whether duplication is OK in tests?
Repository: kconfig
Description
-------
Fix locale-aware reading in KDesktopFile
The underlying KConfig used QLocale::name() for getting the locale
aware part. But this returns "de_DE" while the desktop files store
"de".
In addition it constructs a QLocale object instead of using the
system locale. This has the advantage that the usage of
QLocale::setDafault() gets honored by KConfig.
Diffs (updated)
-----
autotests/kconfigtest.cpp 2b6de0d7d63df6aee69210aa09418628f0b8110a
autotests/kdesktopfiletest.h d57351fd6edcefd6a0efd9126e454546cfc63b55
autotests/kdesktopfiletest.cpp 494a43c0fb5808a261b9beb56cdc4afd8580ab21
src/core/kconfig.cpp ea9746c001e235529a1cdd5865b9e1b5c129b56a
Diff: https://git.reviewboard.kde.org/r/118564/diff/
Testing
-------
added unit test failed before. I'm not 100 % sure whether using bcp47Name is correct.
Thanks,
Martin Gräßlin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140610/45a0d59b/attachment.html>
More information about the Kde-frameworks-devel
mailing list