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

addToClockTime noTimeDiff is not the identity function #18

Open
thielema opened this issue Aug 13, 2024 · 0 comments
Open

addToClockTime noTimeDiff is not the identity function #18

thielema opened this issue Aug 13, 2024 · 0 comments

Comments

@thielema
Copy link

It looks like addToClockTime is badly broken:

*System.Time> let c = TOD ((19078*24+2)*3600+1357) 0
*System.Time> c
Sun Mar 27 04:22:37 CEST 2022
*System.Time> addToClockTime noTimeDiff c
Sun Mar 27 05:22:37 CEST 2022
*System.Time> addToClockTime noTimeDiff $ addToClockTime noTimeDiff c
Sun Mar 27 05:22:37 CEST 2022
@thielema thielema mentioned this issue Aug 13, 2024
Closed
thielema added a commit to thielema/ganeti that referenced this issue Aug 15, 2024
@thielema
Test.Utils.prop_addToClockTime_identity
4fcafe1 
that test should spot the addToClockTime bug faster than prop_timediffAdd
haskell/old-time#18
saschalucas pushed a commit to ganeti/ganeti that referenced this issue Aug 15, 2024
@thielema @saschalucas
Test.Utils.prop_addToClockTime_identity
47a4390 
that test should spot the addToClockTime bug faster than prop_timediffAdd
haskell/old-time#18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@thielema