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