Skip to content

Commit 552cd39

Browse files
committed
Drop Arcs around lightning-liquidity's DefaultTimeProvider
There's no need for these given `DefaultTimeProvider` is a unit struct.
1 parent 603f901 commit 552cd39

File tree

3 files changed

+16
-5
lines changed

3 files changed

+16
-5
lines changed

lightning-background-processor/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1953,7 +1953,7 @@ mod tests {
19531953
Arc<ChannelManager>,
19541954
Arc<dyn Filter + Sync + Send>,
19551955
Arc<Persister>,
1956-
Arc<DefaultTimeProvider>,
1956+
DefaultTimeProvider,
19571957
>;
19581958

19591959
struct Node {

lightning-liquidity/src/manager.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -318,7 +318,7 @@ impl<
318318
CM: Deref + Clone,
319319
C: Deref + Clone,
320320
K: Deref + Clone,
321-
> LiquidityManager<ES, NS, CM, C, K, Arc<DefaultTimeProvider>>
321+
> LiquidityManager<ES, NS, CM, C, K, DefaultTimeProvider>
322322
where
323323
ES::Target: EntropySource,
324324
NS::Target: NodeSigner,
@@ -335,7 +335,6 @@ where
335335
service_config: Option<LiquidityServiceConfig>,
336336
client_config: Option<LiquidityClientConfig>,
337337
) -> Result<Self, lightning::io::Error> {
338-
let time_provider = Arc::new(DefaultTimeProvider);
339338
Self::new_with_custom_time_provider(
340339
entropy_source,
341340
node_signer,
@@ -345,7 +344,7 @@ where
345344
kv_store,
346345
service_config,
347346
client_config,
348-
time_provider,
347+
DefaultTimeProvider,
349348
)
350349
.await
351350
}
@@ -1046,7 +1045,7 @@ impl<
10461045
CM: Deref + Clone,
10471046
C: Deref + Clone,
10481047
KS: Deref + Clone,
1049-
> LiquidityManagerSync<ES, NS, CM, C, KS, Arc<DefaultTimeProvider>>
1048+
> LiquidityManagerSync<ES, NS, CM, C, KS, DefaultTimeProvider>
10501049
where
10511050
ES::Target: EntropySource,
10521051
NS::Target: NodeSigner,

lightning-liquidity/src/utils/time.rs

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,11 @@ pub trait TimeProvider {
1212
}
1313

1414
/// Default time provider using the system clock.
15+
///
16+
/// You likely don't need to use this directly, it is used automatically with
17+
/// [`LiquidityManager::new`]
18+
///
19+
/// [`LiquidityManager::new`]: crate::manager::LiquidityManager::new
1520
#[derive(Clone, Debug)]
1621
#[cfg(feature = "time")]
1722
pub struct DefaultTimeProvider;
@@ -23,3 +28,10 @@ impl TimeProvider for DefaultTimeProvider {
2328
SystemTime::now().duration_since(UNIX_EPOCH).expect("system time before Unix epoch")
2429
}
2530
}
31+
#[cfg(feature = "time")]
32+
impl core::ops::Deref for DefaultTimeProvider {
33+
type Target = Self;
34+
fn deref(&self) -> &Self {
35+
self
36+
}
37+
}

0 commit comments

Comments
 (0)