D20358: Force en locale for propertyinfotest

Stefan BrĂ¼ns noreply at phabricator.kde.org
Sun Apr 7 19:21:38 BST 2019


bruns added inline comments.

INLINE COMMENTS

> propertyinfotest.cpp:51
>      if (!m_useLocalization) {
> -        QLocale().setDefault(QLocale::c());
> +        QLocale().setDefault(QLocale(QStringLiteral("en")));
>      }

This is somewhat strange ...

According to
https://doc.qt.io/qt-5/qlocale.html#Language-enum

> The "C" locale is identical in behavior to English/UnitedStates.

So either "en" is not "en_US", or the Qt documentation is wrong ...

REPOSITORY
  R286 KFileMetaData

REVISION DETAIL
  https://phabricator.kde.org/D20358

To: astippich, bruns
Cc: kde-frameworks-devel, #baloo, gennad, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190407/170b9753/attachment.html>


More information about the Kde-frameworks-devel mailing list