From: Elias Rohrer <dev@tnull.de>
Date: Tue, 9 Jan 2024 08:37:14 +0000 (+0100)
Subject: Feature-gate `time` use also in `ElectrumSyncClient`
X-Git-Tag: v0.0.120~16^2
X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=commitdiff_plain;h=2bd12137a41a8a5206e5c6902e3eac72a9823ef4;p=rust-lightning

Feature-gate `time` use also in `ElectrumSyncClient`

A previous commit introduced the `time` feature to gate the use of
`SystemTime` dependent APIs in `EsploraSyncClient`. It however omitted
doing the same for the Electrum side of things. Here, we address this
oversight.
---

diff --git a/lightning-transaction-sync/src/electrum.rs b/lightning-transaction-sync/src/electrum.rs
index 07e113389..d0c8afef7 100644
--- a/lightning-transaction-sync/src/electrum.rs
+++ b/lightning-transaction-sync/src/electrum.rs
@@ -86,6 +86,7 @@ where
 		let mut sync_state = self.sync_state.lock().unwrap();
 
 		log_trace!(self.logger, "Starting transaction sync.");
+		#[cfg(feature = "time")]
 		let start_time = Instant::now();
 		let mut num_confirmed = 0;
 		let mut num_unconfirmed = 0;
@@ -210,10 +211,15 @@ where
 				sync_state.pending_sync = false;
 			}
 		}
+		#[cfg(feature = "time")]
 		log_debug!(self.logger,
 			"Finished transaction sync at tip {} in {}ms: {} confirmed, {} unconfirmed.",
 			tip_header.block_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_header.block_hash(), num_confirmed, num_unconfirmed);
 		Ok(())
 	}