Skip to content

Replace time.time with time.monotonic to avoid clock drift issues#4492

Merged
Torxed merged 1 commit intoarchlinux:masterfrom
correctmost:cm/remove-time-dot-time
Apr 28, 2026
Merged

Replace time.time with time.monotonic to avoid clock drift issues#4492
Torxed merged 1 commit intoarchlinux:masterfrom
correctmost:cm/remove-time-dot-time

Conversation

@correctmost
Copy link
Copy Markdown
Contributor

PR Description:

Using time.time() can be inaccurate if the system clock gets updated in between calls.

Tests and Checks

  • I tested the ping and pacman-lock paths and didn't see any issues

Using time.time() can be inaccurate if the system clock gets
updated in between calls.
@correctmost correctmost requested a review from Torxed as a code owner April 28, 2026 11:20
Copy link
Copy Markdown
Member

@Torxed Torxed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one really tickles my mind for some reason, nice improvement!

@Torxed Torxed merged commit c4d6885 into archlinux:master Apr 28, 2026
9 checks passed
@correctmost correctmost deleted the cm/remove-time-dot-time branch April 28, 2026 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants