Skip to content

Commit

Permalink
Correct async lightning-background-processor exit check docs
Browse files Browse the repository at this point in the history
The check to stop the background processor is true -> break, not
false -> break. This simply updates the docs and leaves the code
as-is.
  • Loading branch information
TheBlueMatt committed Nov 9, 2022
1 parent 3b2f694 commit 71739db
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions lightning-background-processor/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -358,8 +358,8 @@ macro_rules! define_run_body {
/// Processes background events in a future.
///
/// `sleeper` should return a future which completes in the given amount of time and returns a
/// boolean indicating whether the background processing should continue. Once `sleeper` returns a
/// future which outputs false, the loop will exit and this function's future will complete.
/// boolean indicating whether the background processing should exit. Once `sleeper` returns a
/// future which outputs true, the loop will exit and this function's future will complete.
///
/// See [`BackgroundProcessor::start`] for information on which actions this handles.
#[cfg(feature = "futures")]
Expand Down Expand Up @@ -411,13 +411,13 @@ where
UMH::Target: 'static + CustomMessageHandler,
PS::Target: 'static + Persister<'a, Signer, CW, T, K, F, L, SC>,
{
let mut should_continue = true;
let mut should_break = true;
define_run_body!(persister, event_handler, chain_monitor, channel_manager,
gossip_sync, peer_manager, logger, scorer, should_continue, {
gossip_sync, peer_manager, logger, scorer, should_break, {
select_biased! {
_ = channel_manager.get_persistable_update_future().fuse() => true,
cont = sleeper(Duration::from_millis(100)).fuse() => {
should_continue = cont;
exit = sleeper(Duration::from_millis(100)).fuse() => {
should_break = exit;
false
}
}
Expand Down

0 comments on commit 71739db

Please sign in to comment.