From eef3f7afeb763efa542c4c7c37a993932ffcb291 Mon Sep 17 00:00:00 2001 From: Santiago Carmuega Date: Thu, 9 Dec 2021 07:03:13 -0300 Subject: [PATCH] style: apply fmt to entire workspace --- pallas-alonzo/src/lib.rs | 2 +- pallas-alonzo/src/model.rs | 2 +- pallas-blockfetch/examples/client.rs | 9 ++++--- pallas-blockfetch/src/lib.rs | 31 +++++++++++++++++++----- pallas-chainsync/examples/blocks.rs | 5 +++- pallas-chainsync/src/clients.rs | 17 ++++++++++--- pallas-handshake/examples/node.rs | 3 +-- pallas-handshake/src/n2c.rs | 12 ++++----- pallas-handshake/src/n2n.rs | 4 ++- pallas-localstate/examples/chainpoint.rs | 9 +++---- pallas-localstate/src/codec.rs | 4 ++- pallas-localstate/src/lib.rs | 6 +++-- pallas-localstate/src/queries.rs | 14 +++++------ pallas-machines/src/codec.rs | 2 +- pallas-machines/src/lib.rs | 4 +-- pallas-machines/src/payloads.rs | 11 ++++++--- pallas-multiplexer/examples/listener.rs | 4 +-- pallas-multiplexer/examples/sender.rs | 6 ++--- pallas-multiplexer/src/lib.rs | 13 ++++++++-- pallas-txsubmission/examples/naive.rs | 3 +-- pallas-txsubmission/src/lib.rs | 15 +++++------- pallas/src/ledger/mod.rs | 1 - pallas/src/ouroboros/network.rs | 1 - 23 files changed, 111 insertions(+), 67 deletions(-) diff --git a/pallas-alonzo/src/lib.rs b/pallas-alonzo/src/lib.rs index 91c51d3..2dd6fe7 100644 --- a/pallas-alonzo/src/lib.rs +++ b/pallas-alonzo/src/lib.rs @@ -1,5 +1,5 @@ -mod model; mod framework; +mod model; pub use framework::*; pub use model::*; diff --git a/pallas-alonzo/src/model.rs b/pallas-alonzo/src/model.rs index eb5bb49..e189586 100644 --- a/pallas-alonzo/src/model.rs +++ b/pallas-alonzo/src/model.rs @@ -26,7 +26,7 @@ impl<'b, const N: usize> minicbor::Decode<'b> for SkipCbor { impl minicbor::Encode for SkipCbor { fn encode( &self, - e: &mut minicbor::Encoder, + _e: &mut minicbor::Encoder, ) -> Result<(), minicbor::encode::Error> { todo!() } diff --git a/pallas-blockfetch/examples/client.rs b/pallas-blockfetch/examples/client.rs index b3c868a..5a3e408 100644 --- a/pallas-blockfetch/examples/client.rs +++ b/pallas-blockfetch/examples/client.rs @@ -2,8 +2,11 @@ use net2::TcpStreamExt; use pallas_machines::primitives::Point; use std::net::TcpStream; -use pallas_blockfetch::BlockFetchClient; -use pallas_handshake::{MAINNET_MAGIC, n2n::{Client, VersionTable}}; +use pallas_blockfetch::{BatchClient, NoopObserver}; +use pallas_handshake::{ + n2n::{Client, VersionTable}, + MAINNET_MAGIC, +}; use pallas_machines::run_agent; use pallas_multiplexer::Multiplexer; @@ -37,7 +40,7 @@ fn main() { ); let mut bf_channel = muxer.use_channel(3); - let bf = BlockFetchClient::initial(range); + let bf = BatchClient::initial(range, NoopObserver {}); let bf_last = run_agent(bf, &mut bf_channel); println!("{:?}", bf_last); } diff --git a/pallas-blockfetch/src/lib.rs b/pallas-blockfetch/src/lib.rs index 0ba8d70..78d4b8b 100644 --- a/pallas-blockfetch/src/lib.rs +++ b/pallas-blockfetch/src/lib.rs @@ -94,18 +94,25 @@ pub trait Observer { } } +#[derive(Debug)] pub struct NoopObserver {} impl Observer for NoopObserver {} #[derive(Debug)] -pub struct BatchClient where O: Observer { +pub struct BatchClient +where + O: Observer, +{ pub state: State, pub range: (Point, Point), pub observer: O, } -impl BatchClient where O: Observer { +impl BatchClient +where + O: Observer, +{ pub fn initial(range: (Point, Point), observer: O) -> Self { Self { state: State::Idle, @@ -128,7 +135,10 @@ impl BatchClient where O: Observer { } } -impl Agent for BatchClient where O: Observer { +impl Agent for BatchClient +where + O: Observer, +{ type Message = Message; fn is_done(&self) -> bool { @@ -176,13 +186,19 @@ impl Agent for BatchClient where O: Observer { } #[derive(Debug)] -pub struct OnDemandClient where O: Observer { +pub struct OnDemandClient +where + O: Observer, +{ pub state: State, pub requests: Receiver, pub observer: O, } -impl OnDemandClient where O: Observer { +impl OnDemandClient +where + O: Observer, +{ pub fn initial(requests: Receiver, observer: O) -> Self { Self { state: State::Idle, @@ -207,7 +223,10 @@ impl OnDemandClient where O: Observer { } } -impl Agent for OnDemandClient where O: Observer { +impl Agent for OnDemandClient +where + O: Observer, +{ type Message = Message; // we're never done because we react to external work requests. diff --git a/pallas-chainsync/examples/blocks.rs b/pallas-chainsync/examples/blocks.rs index 8bcf76b..768b5be 100644 --- a/pallas-chainsync/examples/blocks.rs +++ b/pallas-chainsync/examples/blocks.rs @@ -1,5 +1,8 @@ use pallas_chainsync::{ClientConsumer, NoopObserver}; -use pallas_handshake::{MAINNET_MAGIC, n2c::{Client, VersionTable}}; +use pallas_handshake::{ + n2c::{Client, VersionTable}, + MAINNET_MAGIC, +}; use pallas_machines::primitives::Point; use pallas_machines::run_agent; use pallas_multiplexer::Multiplexer; diff --git a/pallas-chainsync/src/clients.rs b/pallas-chainsync/src/clients.rs index a02669c..39f9bb8 100644 --- a/pallas-chainsync/src/clients.rs +++ b/pallas-chainsync/src/clients.rs @@ -9,9 +9,20 @@ use pallas_machines::{ use crate::{BlockBody, Message, State, Tip, WrappedHeader}; /// An observer of chain-sync events sent by the state-machine -pub trait Observer where C: Debug { - fn on_block(&self, cursor: &Option, content: &C) -> Result<(), Box> { - log::debug!("asked to save block content {:?} at cursor {:?}", content, cursor); +pub trait Observer +where + C: Debug, +{ + fn on_block( + &self, + cursor: &Option, + content: &C, + ) -> Result<(), Box> { + log::debug!( + "asked to save block content {:?} at cursor {:?}", + content, + cursor + ); Ok(()) } diff --git a/pallas-handshake/examples/node.rs b/pallas-handshake/examples/node.rs index a933313..0d6d309 100644 --- a/pallas-handshake/examples/node.rs +++ b/pallas-handshake/examples/node.rs @@ -10,8 +10,7 @@ fn main() { env_logger::init(); //let bearer = TcpStream::connect("localhost:6000").unwrap(); - let bearer = - TcpStream::connect("relays-new.cardano-mainnet.iohk.io:3001").unwrap(); + let bearer = TcpStream::connect("relays-new.cardano-mainnet.iohk.io:3001").unwrap(); bearer.set_nodelay(true).unwrap(); bearer.set_keepalive_ms(Some(30_000u32)).unwrap(); diff --git a/pallas-handshake/src/n2c.rs b/pallas-handshake/src/n2c.rs index 237809b..f690757 100644 --- a/pallas-handshake/src/n2c.rs +++ b/pallas-handshake/src/n2c.rs @@ -1,7 +1,9 @@ use core::panic; use std::collections::HashMap; -use pallas_machines::{Agent, CodecError, DecodePayload, EncodePayload, MachineOutput, PayloadDecoder, PayloadEncoder}; +use pallas_machines::{ + Agent, CodecError, DecodePayload, EncodePayload, MachineOutput, PayloadDecoder, PayloadEncoder, +}; use crate::common::{NetworkMagic, RefuseReason, VersionNumber}; @@ -39,11 +41,9 @@ impl VersionTable { } pub fn only_v10(network_magic: u64) -> VersionTable { - let values = vec![ - (PROTOCOL_V10, VersionData(network_magic)), - ] - .into_iter() - .collect::>(); + let values = vec![(PROTOCOL_V10, VersionData(network_magic))] + .into_iter() + .collect::>(); VersionTable { values } } diff --git a/pallas-handshake/src/n2n.rs b/pallas-handshake/src/n2n.rs index 8e4beda..4233c00 100644 --- a/pallas-handshake/src/n2n.rs +++ b/pallas-handshake/src/n2n.rs @@ -1,7 +1,9 @@ use core::panic; use std::collections::HashMap; -use pallas_machines::{Agent, CodecError, DecodePayload, EncodePayload, MachineOutput, PayloadDecoder, PayloadEncoder}; +use pallas_machines::{ + Agent, CodecError, DecodePayload, EncodePayload, MachineOutput, PayloadDecoder, PayloadEncoder, +}; use crate::common::{RefuseReason, VersionNumber}; diff --git a/pallas-localstate/examples/chainpoint.rs b/pallas-localstate/examples/chainpoint.rs index 0b13efd..a922293 100644 --- a/pallas-localstate/examples/chainpoint.rs +++ b/pallas-localstate/examples/chainpoint.rs @@ -1,9 +1,8 @@ -use minicbor::data::Cbor; -use pallas_localstate::queries::RequestV10; -use pallas_localstate::{OneShotClient, queries::QueryV10}; use pallas_handshake::n2c::{Client, VersionTable}; -use pallas_handshake::{MAINNET_MAGIC}; -use pallas_machines::{DecodePayload, EncodePayload, run_agent}; +use pallas_handshake::MAINNET_MAGIC; +use pallas_localstate::queries::RequestV10; +use pallas_localstate::{queries::QueryV10, OneShotClient}; +use pallas_machines::run_agent; use pallas_multiplexer::Multiplexer; use std::os::unix::net::UnixStream; diff --git a/pallas-localstate/src/codec.rs b/pallas-localstate/src/codec.rs index 729961b..7de4c7f 100644 --- a/pallas-localstate/src/codec.rs +++ b/pallas-localstate/src/codec.rs @@ -21,7 +21,9 @@ impl DecodePayload for AcquireFailure { match code { 0 => Ok(AcquireFailure::PointTooOld), 1 => Ok(AcquireFailure::PointNotInChain), - _ => Err(Box::new(CodecError::UnexpectedCbor("can't infer acquire failure from variant id"))), + _ => Err(Box::new(CodecError::UnexpectedCbor( + "can't infer acquire failure from variant id", + ))), } } } diff --git a/pallas-localstate/src/lib.rs b/pallas-localstate/src/lib.rs index 750a1c8..869b4f6 100644 --- a/pallas-localstate/src/lib.rs +++ b/pallas-localstate/src/lib.rs @@ -1,11 +1,13 @@ -pub mod queries; mod codec; +pub mod queries; use std::fmt::Debug; use log::debug; -use pallas_machines::{Agent, DecodePayload, EncodePayload, MachineError, MachineOutput, Transition, primitives::Point}; +use pallas_machines::{ + primitives::Point, Agent, DecodePayload, EncodePayload, MachineError, MachineOutput, Transition, +}; #[derive(Debug, PartialEq, Clone)] pub enum State { diff --git a/pallas-localstate/src/queries.rs b/pallas-localstate/src/queries.rs index fbb902f..178bdba 100644 --- a/pallas-localstate/src/queries.rs +++ b/pallas-localstate/src/queries.rs @@ -1,5 +1,5 @@ -use minicbor::{Decoder, data::Cbor}; -use pallas_machines::{DecodePayload, EncodePayload, PayloadDecoder, primitives::Point}; +use minicbor::{data::Cbor, Decoder}; +use pallas_machines::{primitives::Point, DecodePayload, EncodePayload, PayloadDecoder}; use super::Query; @@ -20,10 +20,8 @@ impl EncodePayload for RequestV10 { e: &mut pallas_machines::PayloadEncoder, ) -> Result<(), Box> { match self { - Self::BlockQuery(block_query) => { - e.u16(0)?; - e.array(0)?; - Ok(()) + Self::BlockQuery(..) => { + todo!() } Self::GetSystemStart => { e.u16(1)?; @@ -43,7 +41,7 @@ impl EncodePayload for RequestV10 { impl DecodePayload for RequestV10 { fn decode_payload( - d: &mut pallas_machines::PayloadDecoder, + _d: &mut pallas_machines::PayloadDecoder, ) -> Result> { todo!() } @@ -55,7 +53,7 @@ pub struct GenericResponse(Vec); impl EncodePayload for GenericResponse { fn encode_payload( &self, - e: &mut pallas_machines::PayloadEncoder, + _e: &mut pallas_machines::PayloadEncoder, ) -> Result<(), Box> { todo!() } diff --git a/pallas-machines/src/codec.rs b/pallas-machines/src/codec.rs index 4f8f68f..cbc1d2e 100644 --- a/pallas-machines/src/codec.rs +++ b/pallas-machines/src/codec.rs @@ -1,5 +1,5 @@ -use super::primitives::*; use super::payloads::*; +use super::primitives::*; impl EncodePayload for Point { fn encode_payload(&self, e: &mut PayloadEncoder) -> Result<(), Box> { diff --git a/pallas-machines/src/lib.rs b/pallas-machines/src/lib.rs index dbde2df..383f75c 100644 --- a/pallas-machines/src/lib.rs +++ b/pallas-machines/src/lib.rs @@ -1,12 +1,12 @@ +mod codec; mod payloads; pub mod primitives; -mod codec; use log::{debug, trace}; use pallas_multiplexer::{Channel, Payload}; use std::borrow::Borrow; use std::fmt::{Debug, Display}; -use std::sync::mpsc::{Sender}; +use std::sync::mpsc::Sender; pub use payloads::*; diff --git a/pallas-machines/src/payloads.rs b/pallas-machines/src/payloads.rs index cd1b973..0ea4285 100644 --- a/pallas-machines/src/payloads.rs +++ b/pallas-machines/src/payloads.rs @@ -2,8 +2,11 @@ use super::*; use log::{debug, warn}; use minicbor::{Decoder, Encoder}; -use std::{ops::{Deref, DerefMut}, sync::mpsc::Receiver}; use pallas_multiplexer::Payload; +use std::{ + ops::{Deref, DerefMut}, + sync::mpsc::Receiver, +}; pub struct PayloadEncoder<'a>(Encoder<&'a mut Vec>); @@ -22,7 +25,10 @@ impl<'a> DerefMut for PayloadEncoder<'a> { } impl<'a> PayloadEncoder<'a> { - pub fn encode_payload(&mut self, t: &T)->Result<(), Box> { + pub fn encode_payload( + &mut self, + t: &T, + ) -> Result<(), Box> { t.encode_payload(self) } } @@ -95,7 +101,6 @@ where } } - pub trait DecodePayload: Sized { fn decode_payload(d: &mut PayloadDecoder) -> Result>; } diff --git a/pallas-multiplexer/examples/listener.rs b/pallas-multiplexer/examples/listener.rs index ecdb3fd..cf531a8 100644 --- a/pallas-multiplexer/examples/listener.rs +++ b/pallas-multiplexer/examples/listener.rs @@ -1,4 +1,4 @@ -use std::{net::TcpListener, os::unix::net::UnixListener, thread, time::Duration}; +use std::{os::unix::net::UnixListener, thread, time::Duration}; use pallas_multiplexer::{Channel, Multiplexer}; @@ -15,7 +15,7 @@ fn main() { for protocol in PROTOCOLS { let handle = muxer.use_channel(protocol); - + thread::spawn(move || { let Channel(_, rx) = handle; diff --git a/pallas-multiplexer/examples/sender.rs b/pallas-multiplexer/examples/sender.rs index a04d892..03af24f 100644 --- a/pallas-multiplexer/examples/sender.rs +++ b/pallas-multiplexer/examples/sender.rs @@ -1,4 +1,4 @@ -use std::{net::TcpStream, os::unix::net::UnixStream, thread, time::Duration}; +use std::{os::unix::net::UnixStream, thread, time::Duration}; use pallas_multiplexer::{Channel, Multiplexer}; @@ -20,9 +20,7 @@ fn main() { loop { let payload = vec![1; 65545]; tx.send(payload).unwrap(); - thread::sleep(Duration::from_millis( - 50u64 + (protocol as u64 * 10u64), - )); + thread::sleep(Duration::from_millis(50u64 + (protocol as u64 * 10u64))); } }); } diff --git a/pallas-multiplexer/src/lib.rs b/pallas-multiplexer/src/lib.rs index 16399dc..70d0cff 100644 --- a/pallas-multiplexer/src/lib.rs +++ b/pallas-multiplexer/src/lib.rs @@ -1,6 +1,12 @@ mod bearers; -use std::{collections::HashMap, io::{Read, Write}, sync::mpsc::{self, Receiver, Sender, TryRecvError}, thread::{self, JoinHandle}, time::{Duration, Instant}}; +use std::{ + collections::HashMap, + io::{Read, Write}, + sync::mpsc::{self, Receiver, Sender, TryRecvError}, + thread::{self, JoinHandle}, + time::{Duration, Instant}, +}; use log::{debug, error, trace, warn}; @@ -115,7 +121,10 @@ pub struct Multiplexer { } impl Multiplexer { - pub fn setup(bearer: TBearer, protocols: &[u16]) -> Result> + pub fn setup( + bearer: TBearer, + protocols: &[u16], + ) -> Result> where TBearer: Bearer + 'static, { diff --git a/pallas-txsubmission/examples/naive.rs b/pallas-txsubmission/examples/naive.rs index 2449d2a..77d7c06 100644 --- a/pallas-txsubmission/examples/naive.rs +++ b/pallas-txsubmission/examples/naive.rs @@ -1,11 +1,11 @@ use net2::TcpStreamExt; use std::net::TcpStream; -use pallas_txsubmission::{NaiveProvider}; use pallas_handshake::n2c::{Client, VersionTable}; use pallas_handshake::MAINNET_MAGIC; use pallas_machines::run_agent; use pallas_multiplexer::Multiplexer; +use pallas_txsubmission::NaiveProvider; fn main() { env_logger::init(); @@ -23,7 +23,6 @@ fn main() { let last = run_agent(Client::initial(versions), &mut hs_channel).unwrap(); println!("{:?}", last); - let mut ts_channel = muxer.use_channel(4); let ts = NaiveProvider::initial(vec![]); let ts = run_agent(ts, &mut ts_channel).unwrap(); diff --git a/pallas-txsubmission/src/lib.rs b/pallas-txsubmission/src/lib.rs index 0d6203b..005134c 100644 --- a/pallas-txsubmission/src/lib.rs +++ b/pallas-txsubmission/src/lib.rs @@ -3,7 +3,10 @@ use std::fmt::Debug; use itertools::Itertools; use log::debug; -use pallas_machines::{Agent, CodecError, DecodePayload, EncodePayload, MachineError, MachineOutput, PayloadDecoder, PayloadEncoder, Transition}; +use pallas_machines::{ + Agent, CodecError, DecodePayload, EncodePayload, MachineError, MachineOutput, PayloadDecoder, + PayloadEncoder, Transition, +}; #[derive(Debug, PartialEq, Clone)] pub enum State { @@ -238,14 +241,8 @@ impl NaiveProvider { }) } - fn on_txs_request( - self, - requested_txs: Vec, - ) -> Transition { - debug!( - "new txs request {:?}", - requested_txs, - ); + fn on_txs_request(self, requested_txs: Vec) -> Transition { + debug!("new txs request {:?}", requested_txs,); Ok(Self { state: State::Idle, diff --git a/pallas/src/ledger/mod.rs b/pallas/src/ledger/mod.rs index 3352b92..03c0952 100644 --- a/pallas/src/ledger/mod.rs +++ b/pallas/src/ledger/mod.rs @@ -1,3 +1,2 @@ - #[doc(inline)] pub use pallas_alonzo as alonzo; diff --git a/pallas/src/ouroboros/network.rs b/pallas/src/ouroboros/network.rs index ceeef8e..1b1e829 100644 --- a/pallas/src/ouroboros/network.rs +++ b/pallas/src/ouroboros/network.rs @@ -1,4 +1,3 @@ - #[doc(inline)] pub use pallas_multiplexer as multiplexer;