Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Remove time-utils library #10723

Closed
niklasad1 opened this issue Jun 5, 2019 · 1 comment
Closed

Remove time-utils library #10723

niklasad1 opened this issue Jun 5, 2019 · 1 comment
Labels
P5-sometimesoon 🌲 Issue is worth doing soon. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow.

Comments

@niklasad1
Copy link
Collaborator

The time-utils library was temporary introduced for checked operations on SystemTime, until it was stabilized in the standard library.

The checked operations were stabilized in Rust 1.34 and latest stable is at 1.35
@niklasad1 niklasad1 added P5-sometimesoon 🌲 Issue is worth doing soon. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow. labels Jun 5, 2019
@niklasad1
Copy link
Collaborator Author

Not valid anymore, we will use CheckedSystemTime:checked_xx to limit biggest timestamp to be i32::max_value() to circumvent platform dependent potential consensus issues!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
P5-sometimesoon 🌲 Issue is worth doing soon. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@niklasad1