Skip to content

Commit 1900493

Browse files
committed
satisfy the clippy
1 parent 318d11e commit 1900493

3 files changed

Lines changed: 13 additions & 3 deletions

File tree

crates/amaru-consensus/src/chain_forward.rs

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,9 @@ impl ForwardStage {
6969
network_magic: u64,
7070
listen_address: &str,
7171
) -> Self {
72-
let runtime = AcTokio::new("consensus.forward", 1).unwrap();
72+
#[allow(clippy::expect_used)]
73+
let runtime =
74+
AcTokio::new("consensus.forward", 1).expect("failed to create AcTokio runtime");
7375
Self {
7476
store,
7577
upstream: Default::default(),
@@ -105,7 +107,9 @@ impl gasket::framework::Worker<ForwardStage> for Worker {
105107
let server = TcpListener::bind(&stage.listen_address).await.or_panic()?;
106108
if let Some(downstream) = &stage.downstream {
107109
tracing::debug!("sending listening event");
108-
downstream.send(ForwardEvent::Listening(server.local_addr().unwrap().port()));
110+
downstream.send(ForwardEvent::Listening(
111+
server.local_addr().or_panic()?.port(),
112+
));
109113
}
110114

111115
let (tx, incoming_peers) = mpsc::channel(10);
@@ -247,6 +251,7 @@ impl gasket::framework::Worker<ForwardStage> for Worker {
247251
}
248252
}
249253

254+
#[allow(clippy::large_enum_variant)]
250255
enum ClientMsg {
251256
Peer(PeerServer, Tip),
252257
Op(ClientOp),

crates/amaru-consensus/src/chain_forward/client_protocol.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ pub async fn client_protocols(
6868
Ok(())
6969
}
7070

71+
#[allow(clippy::large_enum_variant)]
7172
pub enum ChainSyncMsg {
7273
/// An operation coming in from the ledger
7374
Op(ClientOp),

crates/amaru-consensus/src/chain_forward/client_state.rs

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ use pallas_network::miniprotocols::{chainsync::Tip, Point};
55
use std::{collections::VecDeque, sync::Arc};
66
use tokio::sync::Mutex;
77

8+
#[allow(clippy::large_enum_variant)]
89
#[derive(Debug, Clone, PartialEq, Eq)]
910
pub(super) enum ClientOp {
1011
Backward(Point),
@@ -47,7 +48,10 @@ impl ClientState {
4748
match op {
4849
ClientOp::Backward(point) => {
4950
let store = self.store.lock().await;
50-
let header = store.load_header(&hash_point(point)).unwrap();
51+
#[allow(clippy::expect_used)]
52+
let header = store
53+
.load_header(&hash_point(point))
54+
.expect("rollback point was not in store");
5155
Tip(point.clone(), header.block_height())
5256
}
5357
ClientOp::Forward(header) => {

0 commit comments

Comments
 (0)