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.

Sign up for GitHub

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

Jump to bottom

chrono::DurationRound is not TZ aware #495

Closed
SimGer opened this issue Oct 28, 2020 · 0 comments · Fixed by #508
Closed

chrono::DurationRound is not TZ aware #495

SimGer opened this issue Oct 28, 2020 · 0 comments · Fixed by #508
Labels
bug

Comments

@SimGer
Copy link

SimGer commented Oct 28, 2020

The functions implemented in the chrono::DurationRound trait return wrong values when a time zone is involved.
The TZ offset "leaks" into the time value.

Example on Rust Playground.

If this is expected it should be documented, I think.
robyoung added a commit to robyoung/chrono that referenced this issue Nov 30, 2020
@robyoung
Fix chrono::DurationRound is not TZ aware
74406a5 
Fixes chronotope#495
@robyoung robyoung mentioned this issue Nov 30, 2020
Merged
pickfire pushed a commit to pickfire/chrono that referenced this issue Jan 22, 2022
@robyoung @pickfire
Fix chrono::DurationRound is not TZ aware
67627f8 
Fixes chronotope#495
botahamec pushed a commit to botahamec/chrono that referenced this issue May 26, 2022
@robyoung @botahamec
Fix chrono::DurationRound is not TZ aware
db02854 
Fixes chronotope#495
pickfire pushed a commit to pickfire/chrono that referenced this issue Jul 5, 2022
@robyoung @pickfire
Fix chrono::DurationRound is not TZ aware
e6f4385 
Fixes chronotope#495
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix chrono::DurationRound is not TZ aware robyoung/chrono
2 participants
@quodlibetor @SimGer