----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118692/ -----------------------------------------------------------
(Updated June 16, 2014, 10:26 a.m.) Review request for KDE Frameworks, David Faure, John Layt, and Oswald Buddenhagen. Changes ------- * dropped the modifier handling as that's not supported by Qt * extended the unit test a little bit * use indexOf instead of converting to QByteArray. 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 (updated) ----- autotests/kdesktopfiletest.cpp 6c3af4c2cd55b9140c0cd48526947431ceb7e798 src/core/kconfig.cpp a2598f8e8fce91a8de3f34b272e15a6c280a50db src/core/kconfigdata.h fdec85dc90467560bd312b72266ef0b3f243d076 src/core/kconfigdata.cpp 109063d65e97bcb7ba08cf6e5a6f3acb885fe845 src/core/kconfigini.cpp a882ee31150658f3e5cfb036362ff0583f71cbd9 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
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
