Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[time.zone.leap.members] update note about leap seconds #6016

Merged
merged 1 commit into from Dec 15, 2022

Conversation

jwakely
Copy link
Member

@jwakely jwakely commented Dec 13, 2022

This note about leap seconds is still true, but should be updated to be more precise. Saying something about 2019 in C++23 seems odd, and will lead readers to wonder if a negative leap seconds was inserted in 2020.

There has been no leap second inserted since 2016, and none will be inserted before 2023-06-28 at the earliest.

https://en.wikipedia.org/wiki/Leap_second

@jensmaurer
Copy link
Member

Somehow I feel we should change the note so that it doesn't become invalid for every standards release. But the proposed adjustment is better than no change at all.

@jwakely
Copy link
Member Author

jwakely commented Dec 13, 2022

It looks like the next leap second might be negative, so we would need to either remove or change it to a definite statement about the past. But there might be three standard releases before that actually happens.

@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 15, 2022

I thought leap seconds are now abolished?

@tkoeppe tkoeppe merged commit 5d43f01 into cplusplus:main Dec 15, 2022
@jwakely
Copy link
Member Author

jwakely commented Dec 16, 2022

There is agreement to stop adding them after 2035, but that still has to be ratified by the ITU. Even if the ITU agrees, there could be more leap seconds added before 2035.

https://www.nature.com/articles/d41586-022-03783-5

@jwakely jwakely deleted the leaps branch December 16, 2022 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants