- assumption that single-zone countries do not have description is no longer correct; do not try to optimize this case as it's only going to make the code more confusing and we now have menus with a single zone selection because of this
- remove the single-country continent short cut, it also only serves to confuse users as we now have such a continent
- instead add a single-zone contry short cut (see above), now all single-zone countries fall here
- use the @# continent overrides that zone1970.tab introduces (this is visible at least fixing Iceland being currently listed under Africa)
- add Arctic Ocean "continent" coming only from the overrides at the moment
- update baseline with the changes
Details
Details
- Reviewers
philip bapt - Commits
- rGcfd0217aa9a9: tzsetup: adopt zone1970.tab changes
The only really possible testing here is checking that tzsetup can parse zone1970.tab and provided selections still make sense; the latter should be covered by D39634.
Diff Detail
Diff Detail
- Repository
- rG FreeBSD src repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
LGTM, I added @philip as he has been updating the tzdata for a while now, so I might have an opinion
Comment Actions
Thank you very much!
I gave this a cursory test, and it looks like it works. Thank you for doing this work.