Skip to content

[4/6] Replace continuous polling with a configurable delay - #4

Merged
GlenTiki merged 1 commit into
masterfrom
modernize/04-polling
Oct 8, 2026
Merged

GlenTiki merged 1 commit into
masterfrom
modernize/04-polling

Conversation

@GlenTiki

@GlenTiki GlenTiki commented Oct 8, 2026

Copy link
Copy Markdown
Owner

I replace immediate polling with a 10 ms default delay and a configurable interval. Initial checks and reset remain synchronous. I validate the timer range and use controlled timers to test polling and cancellation.

This is PR 4 of 6 in the modernisation stack. It depends on #3. Review and merge the stack in order.

Dependency removal and the major-version package metadata are left to the next PR.

@GlenTiki
GlenTiki force-pushed the modernize/04-polling branch from 168f1fd to dace72c Compare October 8, 2026 13:05
@GlenTiki
GlenTiki added this pull request to stack #7 October 8, 2026 13:12
@GlenTiki
GlenTiki force-pushed the modernize/04-polling branch 2 times, most recently from 8f8e4a0 to 9888d70 Compare October 8, 2026 13:14
Base automatically changed from modernize/03-validation to master October 8, 2026 13:15
@GlenTiki
GlenTiki force-pushed the modernize/04-polling branch from 9888d70 to ad18c46 Compare October 8, 2026 13:15
@GlenTiki
GlenTiki merged commit 5b99885 into master Oct 8, 2026
3 checks passed
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.

1 participant