Skip to content

Commit

Permalink
Don't call system time in unless feature enabled
Browse files Browse the repository at this point in the history
  • Loading branch information
benthecarman committed Dec 19, 2023
1 parent 4deb263 commit cb85608
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
3 changes: 2 additions & 1 deletion lightning-transaction-sync/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ all-features = true
rustdoc-args = ["--cfg", "docsrs"]

[features]
default = []
default = ["time"]
time = []
esplora-async = ["async-interface", "esplora-client/async", "futures"]
esplora-async-https = ["esplora-async", "esplora-client/async-https-rustls"]
esplora-blocking = ["esplora-client/blocking"]
Expand Down
8 changes: 6 additions & 2 deletions lightning-transaction-sync/src/esplora.rs
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ use esplora_client::r#async::AsyncClient;
#[cfg(not(feature = "async-interface"))]
use esplora_client::blocking::BlockingClient;

use std::time::Instant;
use std::collections::HashSet;
use core::ops::Deref;

Expand Down Expand Up @@ -91,7 +90,8 @@ where
let mut sync_state = self.sync_state.lock().await;

log_trace!(self.logger, "Starting transaction sync.");
let start_time = Instant::now();
#[cfg(feature = "time")]
let start_time = std::time::Instant::now();
let mut num_confirmed = 0;
let mut num_unconfirmed = 0;

Expand Down Expand Up @@ -227,8 +227,12 @@ where
sync_state.pending_sync = false;
}
}
#[cfg(feature = "time")]
log_debug!(self.logger, "Finished transaction sync at tip {} in {}ms: {} confirmed, {} unconfirmed.",
tip_hash, start_time.elapsed().as_millis(), num_confirmed, num_unconfirmed);
#[cfg(not(feature = "time"))]
log_debug!(self.logger, "Finished transaction sync at tip {}: {} confirmed, {} unconfirmed.",
tip_hash, num_confirmed, num_unconfirmed);
Ok(())
}

Expand Down

0 comments on commit cb85608

Please sign in to comment.