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