Review Request 129691: digital-clock: Update iso-3166 codes in timezones

Pino Toscano pino at kde.org
Tue Dec 20 22:46:54 UTC 2016


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



TBH I don't think the Congo and CZ changes are somehow needed, since the current names are already the official names for the countries.


applets/digital-clock/plugin/timezonesi18n.cpp (line 531)
<https://git.reviewboard.kde.org/r/129691/#comment67963>

    This follows the official name already, not the common name, so the change is not needed.



applets/digital-clock/plugin/timezonesi18n.cpp (line 717)
<https://git.reviewboard.kde.org/r/129691/#comment67967>

    Theoretically "Czech Republic" is correct as well, since it's the official name (as also recorded by iso-codes).


- Pino Toscano


On Dec. 20, 2016, 9:57 p.m., Jiri Bohac wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129691/
> -----------------------------------------------------------
> 
> (Updated Dec. 20, 2016, 9:57 p.m.)
> 
> 
> Review request for Plasma and Albert Astals Cid.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Update the list of country names to the current names in iso-codes
> (as output by isoquery -c).
> 
> The motivation is updating the name of Czechia as a followup to
> https://git.reviewboard.kde.org/r/129644/
> As suggested by Albert Astals Cid, fixing the name in i10n is not enough: 
> country names are hardcoded in many places and they need to be fixed separately.
> 
> 
> Diffs
> -----
> 
>   applets/digital-clock/plugin/timezonesi18n.cpp 10650f8 
> 
> Diff: https://git.reviewboard.kde.org/r/129691/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jiri Bohac
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161220/fc47662f/attachment-0001.html>


More information about the Plasma-devel mailing list