Opened 3 years ago

Closed 3 years ago

#5114 enhancement closed fixed (fixed)

LoopingCall should be resetable

Reported by: chris- Owned by: therve
Priority: normal Milestone:
Component: core Keywords: patch, review
Cc: thijs Branch: branches/reset-loop-5114
(diff, github, buildbot, log)
Author: therve Launchpad Bug:

Description

It would be a good thing if the LoopingCall would be "resetable", meaning it should be easy to stop the currently scheduled call and reschedule it with the currently set interval, providing a nicer alternative to stopping the call and starting a new LoopingCall with the exact same settings.
The following patch implements this.

Attachments (1)

reset_loop.patch (1.5 KB) - added by chris- 3 years ago.

Download all attachments as: .zip

Change History (5)

Changed 3 years ago by chris-

comment:1 Changed 3 years ago by thijs

  • Cc thijs added
  • Keywords review added

Adding missing 'review' keyword (see ReviewProcess).

comment:2 Changed 3 years ago by therve

  • Owner set to therve

comment:3 Changed 3 years ago by therve

  • Author set to therve
  • Branch set to branches/reset-loop-5114

(In [31998]) Branching to 'reset-loop-5114'

comment:4 Changed 3 years ago by therve

  • Resolution set to fixed
  • Status changed from new to closed

(In [32000]) Merge reset-loop-5114

Author: chris
Reviewer: therve
Fixes: #5114

Add a reset method to LoopingCall to be able to stop it and reschedule at once.

Note: See TracTickets for help on using tickets.