Run rustfmt with rust 1.38

.travis.yml

@@ -18,7 +18,6 @@ rust:
 matrix:
   allow_failures:
     - rust: beta
-    - rust: nightly
   fast_finish: true
 install:
   - rustup component add rustfmt

beacon_node/network/src/message_handler.rs

@@ -146,15 +146,9 @@ impl<T: BeaconChainTypes + 'static> MessageHandler<T> {
     ) {
         // an error could have occurred.
         match error_response {
-            RPCErrorResponse::InvalidRequest(error) => {
-                warn!(self.log, "Peer indicated invalid request";"peer_id" => format!("{:?}", peer_id), "error" => error.as_string())
-            }
-            RPCErrorResponse::ServerError(error) => {
-                warn!(self.log, "Peer internal server error";"peer_id" => format!("{:?}", peer_id), "error" => error.as_string())
-            }
-            RPCErrorResponse::Unknown(error) => {
-                warn!(self.log, "Unknown peer error";"peer" => format!("{:?}", peer_id), "error" => error.as_string())
-            }
+            RPCErrorResponse::InvalidRequest(error) => warn!(self.log, "Peer indicated invalid request";"peer_id" => format!("{:?}", peer_id), "error" => error.as_string()),
+            RPCErrorResponse::ServerError(error) => warn!(self.log, "Peer internal server error";"peer_id" => format!("{:?}", peer_id), "error" => error.as_string()),
+            RPCErrorResponse::Unknown(error) => warn!(self.log, "Unknown peer error";"peer" => format!("{:?}", peer_id), "error" => error.as_string()),
             RPCErrorResponse::Success(response) => {
                 match response {
                     RPCResponse::Hello(hello_message) => {

eth2/types/src/attestation.rs

@@ -58,5 +58,4 @@ mod tests {
     use crate::*;
 
     ssz_tests!(Attestation<MainnetEthSpec>);
-
 }

eth2/types/src/attestation_data_and_custody_bit.rs

@@ -19,5 +19,4 @@ mod test {
     use super::*;
 
     ssz_tests!(AttestationDataAndCustodyBit);
-
 }

eth2/types/src/attester_slashing.rs

@@ -21,5 +21,4 @@ mod tests {
     use crate::*;
 
     ssz_tests!(AttesterSlashing<MainnetEthSpec>);
-
 }

eth2/types/src/checkpoint.rs

@@ -32,5 +32,4 @@ mod tests {
     use super::*;
 
     ssz_tests!(Checkpoint);
-
 }

eth2/types/src/crosslink.rs

@@ -37,5 +37,4 @@ mod tests {
     use super::*;
 
     ssz_tests!(Crosslink);
-
 }

eth2/types/src/deposit.rs

@@ -21,5 +21,4 @@ mod tests {
     use super::*;
 
     ssz_tests!(Deposit);
-
 }

eth2/types/src/deposit_data.rs

@@ -55,5 +55,4 @@ mod tests {
     use super::*;
 
     ssz_tests!(DepositData);
-
 }

eth2/types/src/eth1_data.rs

@@ -23,5 +23,4 @@ mod tests {
     use super::*;
 
     ssz_tests!(Eth1Data);
-
 }

eth2/types/src/historical_batch.rs

@@ -23,5 +23,4 @@ mod tests {
     pub type FoundationHistoricalBatch = HistoricalBatch<MainnetEthSpec>;
 
     ssz_tests!(FoundationHistoricalBatch);
-
 }

eth2/types/src/pending_attestation.rs

@@ -23,5 +23,4 @@ mod tests {
     use crate::*;
 
     ssz_tests!(PendingAttestation<MainnetEthSpec>);
-
 }

eth2/types/src/proposer_slashing.rs

@@ -21,5 +21,4 @@ mod tests {
     use super::*;
 
     ssz_tests!(ProposerSlashing);
-
 }

eth2/types/src/transfer.rs

@@ -42,5 +42,4 @@ mod tests {
     use super::*;
 
     ssz_tests!(Transfer);
-
 }

eth2/types/src/validator.rs

@@ -117,5 +117,4 @@ mod tests {
     }
 
     ssz_tests!(Validator);
-
 }

eth2/types/src/voluntary_exit.rs

@@ -35,5 +35,4 @@ mod tests {
     use super::*;
 
     ssz_tests!(VoluntaryExit);
-
 }

validator_client/src/attestation_producer/mod.rs

@@ -57,21 +57,11 @@ impl<'a, B: BeaconNodeAttestation, S: Signer, E: EthSpec> AttestationProducer<'a
                 "slot" => slot,
             ),
             Err(e) => error!(log, "Attestation production error"; "Error" => format!("{:?}", e)),
-            Ok(ValidatorEvent::SignerRejection(_slot)) => {
-                error!(log, "Attestation production error"; "Error" => "Signer could not sign the attestation".to_string())
-            }
-            Ok(ValidatorEvent::IndexedAttestationNotProduced(_slot)) => {
-                error!(log, "Attestation production error"; "Error" => "Rejected the attestation as it could have been slashed".to_string())
-            }
-            Ok(ValidatorEvent::PublishAttestationFailed) => {
-                error!(log, "Attestation production error"; "Error" => "Beacon node was unable to publish an attestation".to_string())
-            }
-            Ok(ValidatorEvent::InvalidAttestation) => {
-                error!(log, "Attestation production error"; "Error" => "The signed attestation was invalid".to_string())
-            }
-            Ok(v) => {
-                warn!(log, "Unknown result for attestation production"; "Error" => format!("{:?}",v))
-            }
+            Ok(ValidatorEvent::SignerRejection(_slot)) => error!(log, "Attestation production error"; "Error" => "Signer could not sign the attestation".to_string()),
+            Ok(ValidatorEvent::IndexedAttestationNotProduced(_slot)) => error!(log, "Attestation production error"; "Error" => "Rejected the attestation as it could have been slashed".to_string()),
+            Ok(ValidatorEvent::PublishAttestationFailed) => error!(log, "Attestation production error"; "Error" => "Beacon node was unable to publish an attestation".to_string()),
+            Ok(ValidatorEvent::InvalidAttestation) => error!(log, "Attestation production error"; "Error" => "The signed attestation was invalid".to_string()),
+            Ok(v) => warn!(log, "Unknown result for attestation production"; "Error" => format!("{:?}",v)),
         }
     }
 

validator_client/src/block_producer/mod.rs

@@ -69,18 +69,10 @@ impl<'a, B: BeaconNodeBlock, S: Signer, E: EthSpec> BlockProducer<'a, B, S, E> {
                 "slot" => slot,
             ),
             Err(e) => error!(self.log, "Block production error"; "Error" => format!("{:?}", e)),
-            Ok(ValidatorEvent::SignerRejection(_slot)) => {
-                error!(self.log, "Block production error"; "Error" => "Signer Could not sign the block".to_string())
-            }
-            Ok(ValidatorEvent::SlashableBlockNotProduced(_slot)) => {
-                error!(self.log, "Block production error"; "Error" => "Rejected the block as it could have been slashed".to_string())
-            }
-            Ok(ValidatorEvent::BeaconNodeUnableToProduceBlock(_slot)) => {
-                error!(self.log, "Block production error"; "Error" => "Beacon node was unable to produce a block".to_string())
-            }
-            Ok(v) => {
-                warn!(self.log, "Unknown result for block production"; "Error" => format!("{:?}",v))
-            }
+            Ok(ValidatorEvent::SignerRejection(_slot)) => error!(self.log, "Block production error"; "Error" => "Signer Could not sign the block".to_string()),
+            Ok(ValidatorEvent::SlashableBlockNotProduced(_slot)) => error!(self.log, "Block production error"; "Error" => "Rejected the block as it could have been slashed".to_string()),
+            Ok(ValidatorEvent::BeaconNodeUnableToProduceBlock(_slot)) => error!(self.log, "Block production error"; "Error" => "Beacon node was unable to produce a block".to_string()),
+            Ok(v) => warn!(self.log, "Unknown result for block production"; "Error" => format!("{:?}",v)),
         }
     }
 

validator_client/src/duties/mod.rs

@@ -77,12 +77,8 @@ impl<U: BeaconNodeDuties, S: Signer + Display> DutiesManager<U, S> {
     pub fn run_update(&self, epoch: Epoch, log: slog::Logger) -> Result<Async<()>, ()> {
         match self.update(epoch) {
             Err(error) => error!(log, "Epoch duties poll error"; "error" => format!("{:?}", error)),
-            Ok(UpdateOutcome::NoChange(epoch)) => {
-                debug!(log, "No change in duties"; "epoch" => epoch)
-            }
-            Ok(UpdateOutcome::DutiesChanged(epoch, duties)) => {
-                info!(log, "Duties changed (potential re-org)"; "epoch" => epoch, "duties" => format!("{:?}", duties))
-            }
+            Ok(UpdateOutcome::NoChange(epoch)) => debug!(log, "No change in duties"; "epoch" => epoch),
+            Ok(UpdateOutcome::DutiesChanged(epoch, duties)) => info!(log, "Duties changed (potential re-org)"; "epoch" => epoch, "duties" => format!("{:?}", duties)),
             Ok(UpdateOutcome::NewDuties(epoch, duties)) => {
                 info!(log, "New duties obtained"; "epoch" => epoch);
                 print_duties(&log, duties);