D10446: Add KLanguageName
Albert Astals Cid
noreply at phabricator.kde.org
Mon Jan 7 23:10:57 GMT 2019
aacid added inline comments.
INLINE COMMENTS
> klanguagename.cpp:30
> + const QStringList parts = QLocale().name().split(QChar('_'));
> + return nameForCodeInLocale(code, parts.at(0));
> +}
should the second param be code too?
I mean if we read the docs we say both are code (ISO 639-1), so both should be the same and not only the first part?
REPOSITORY
R265 KConfigWidgets
REVISION DETAIL
https://phabricator.kde.org/D10446
To: aacid
Cc: hein, kde-frameworks-devel, sitter, markg, apol, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190107/bc06e3ef/attachment.html>
More information about the Kde-frameworks-devel
mailing list