Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merge v1.2.1 into v1.x.x #336

Merged
merged 6 commits into from
Nov 19, 2024
Merged

Merge v1.2.1 into v1.x.x #336

merged 6 commits into from
Nov 19, 2024

Conversation

llucax
Copy link
Contributor

@llucax llucax commented Nov 19, 2024

Fix memory leak in the timer.

shsms and others added 6 commits November 15, 2024 18:21
We were creating two futures, but only awaiting until the one of them
finished and not canceling the second.

Because `reset()` was never called, we were just creating more and
more futures that would never complete.

This change fixes this by canceling the second task immediately after
the first one is complete.

Signed-off-by: Sahas Subramanian <sahas.subramanian@proton.me>
Signed-off-by: Sahas Subramanian <sahas.subramanian@proton.me>
We were creating two futures, but only awaiting until the one of them
finished and not canceling the second.

Because `reset()` was never called, we were just creating more and
more futures that would never complete.

This change fixes this by canceling the second task immediately after
the first one is complete.
Signed-off-by: Sahas Subramanian <sahas.subramanian@proton.me>
Fix a memory leak in the timer.
@llucax llucax requested a review from a team as a code owner November 19, 2024 13:03
@llucax llucax changed the title merge v1.2.1 Merge v1.2.1 into v1.x.x Nov 19, 2024
@github-actions github-actions bot added part:docs Affects the documentation part:utilities Affects the utility receivers (`Timer`, `Event`, `FileWatcher`) labels Nov 19, 2024
@llucax llucax requested a review from shsms November 19, 2024 13:04
@llucax llucax self-assigned this Nov 19, 2024
@llucax llucax added this to the v1.3.0 milestone Nov 19, 2024
@llucax llucax added type:bug Something isn't working type:regression Something isn't working, but it was working before and removed type:bug Something isn't working labels Nov 19, 2024
@llucax llucax enabled auto-merge November 19, 2024 13:04
@llucax llucax added this pull request to the merge queue Nov 19, 2024
Merged via the queue into frequenz-floss:v1.x.x with commit b3c98e5 Nov 19, 2024
17 checks passed
@llucax llucax deleted the merge-v1.2.1 branch November 19, 2024 13:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
part:docs Affects the documentation part:utilities Affects the utility receivers (`Timer`, `Event`, `FileWatcher`) type:regression Something isn't working, but it was working before
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants