GetTime
is mockable in tests and system-changeable in production. This should be fine and not lead to issues, but using SteadyClock
is more correct in this context to do an expensive task only so often.
GetTime
is mockable in tests and system-changeable in production. This should be fine and not lead to issues, but using SteadyClock
is more correct in this context to do an expensive task only so often.
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
For detailed information about the code coverage, see the test coverage report.
See the guideline for information on the review process.
Type | Reviewers |
---|---|
ACK | sipa, TheCharlatan |
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.