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

Add strong types for time #320

Closed
Tracked by #199
PatrickKa opened this issue Aug 12, 2024 · 0 comments · Fixed by #319
Closed
Tracked by #199

Add strong types for time #320

PatrickKa opened this issue Aug 12, 2024 · 0 comments · Fixed by #319
Assignees

Comments

@PatrickKa
Copy link
Contributor

PatrickKa commented Aug 12, 2024

Description

We have two times: "Rodos time" and "real time". The first is the time since the last reset in nanoseconds and has type int64_t. The second is our best guess for the current UTC, given as a UNIX timestamp in seconds with type int32_t. Both times need a strong type for time points, but I think that only the Rodos time needs a duration type. Those three types should be called RodosTime, Duration and RealTime.

Of course those types should also be used, e.g., for the timeout parameters of the SPI communication functions or the start times of the EDU programs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants