Skip to content

Allow subsequent calls to Start and Stop#177

Merged
davseby merged 3 commits intov3from
start-stop-loop
May 25, 2025
Merged

Allow subsequent calls to Start and Stop#177
davseby merged 3 commits intov3from
start-stop-loop

Conversation

@davseby
Copy link
Copy Markdown
Contributor

@davseby davseby commented May 25, 2025

Added a new flag to support calling stopped cache cleanup procedures as well as a mutex to control it. Now:

  1. Start method will only run once even with the subsequent calls to it.
  2. Stop will never block, in case the Start was not called or Stop was called before, it will return as the cache cleanup process is not running.

Closes #175

@davseby davseby requested a review from swithek May 25, 2025 11:57
@davseby davseby self-assigned this May 25, 2025
@davseby davseby changed the title Allow subsequent calls to Start and Stop Allow subsequent calls to Start and Stop May 25, 2025
@coveralls
Copy link
Copy Markdown

coveralls commented May 25, 2025

Pull Request Test Coverage Report for Build 15237732703

Details

  • 17 of 17 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.006%) to 99.717%

Totals Coverage Status
Change from base Build 15237488044: 0.006%
Covered Lines: 704
Relevant Lines: 706

💛 - Coveralls

@davseby davseby merged commit 1fa46c2 into v3 May 25, 2025
2 checks passed
@davseby davseby deleted the start-stop-loop branch May 25, 2025 12:14
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.

Stop blocks if called more than once

3 participants