Review Request 126879: [digital-clock] Use iso-codes for country names

David Edmundson david at davidedmundson.co.uk
Sun Jan 24 22:39:14 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126879/#review91552
-----------------------------------------------------------



+1


applets/digital-clock/plugin/timezonesi18n.cpp (line 517)
<https://git.reviewboard.kde.org/r/126879/#comment62562>

    may as well keep the comment from the i18n?


- David Edmundson


On Jan. 24, 2016, 9:58 p.m., Pino Toscano wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126879/
> -----------------------------------------------------------
> 
> (Updated Jan. 24, 2016, 9:58 p.m.)
> 
> 
> Review request for Localization and Translation (l10n), Plasma and John Layt.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Instead of an hardcoded list of translations of country names, look them up in the iso_3166 domain, provided by iso-codes.  The names of countries are changes to their ISO 3166 versions in iso-codes.
> 
> Look for iso-codes in cmake, but just as indication of it as runtime dependency.
> 
> 
> Diffs
> -----
> 
>   applets/digital-clock/plugin/CMakeLists.txt c22c82a75fe6efecc4a249ebd7e045a6f0385ac5 
>   applets/digital-clock/plugin/timezonesi18n.cpp 6bb465e68bfba4695c79898d630dec5a7075a058 
>   cmake/FindIsoCodes.cmake PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/126879/diff/
> 
> 
> Testing
> -------
> 
> With Plasma 5.3 (but this code is the same as in master), the configuration of timezoned in the digital clock is translated according to iso-codes.
> 
> 
> Thanks,
> 
> Pino Toscano
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160124/08c529ae/attachment.html>


More information about the Plasma-devel mailing list