D22069: Localize long number strings
Chusslove Illich
noreply at phabricator.kde.org
Tue Jun 25 09:01:33 BST 2019
ilic added inline comments.
INLINE COMMENTS
> klocalizedstringtest.cpp:222
> +
> + QLocale::setDefault(QLocale(QLocale::French));
> + QCOMPARE(ki18n("%1").subs(4.2, 5, 'f', 2).toString(),
The French locale is already being set in line 45 using setlocale, for which the system has to have fr_FR.utf8 glibc locale installed. Any further setting up of locale details, if needed, should happen in lines 70-76, which are currently commented out with "until locale system is ready" (and if not needed, this whole commented out part should be deleted).
> klocalizedstringtest.cpp:226
> +
> + QLocale::setDefault(QLocale(QLocale::English, QLocale::UnitedStates));
> QCOMPARE(ki18n("%1").subs(4.2, 5, 'f', 2).toString(),
The tests work with French locale, there is no reason introduce English here only. It also clobbers the globaly set locale elements in initTestCase.
REPOSITORY
R249 KI18n
REVISION DETAIL
https://phabricator.kde.org/D22069
To: ngraham, #localization, #frameworks, broulik
Cc: mikeroyal, aspotashev, ilic, kde-frameworks-devel, broulik, LeGast00n, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190625/36549f04/attachment.html>
More information about the Kde-frameworks-devel
mailing list