Random cleanup

core/src/blocktree_processor.rs

@@ -86,16 +86,6 @@ pub fn process_entries(bank: &Bank, entries: &[Entry]) -> Result<()> {
     par_process_entries(bank, entries)
 }
 
-/// Process an ordered list of entries, populating a circular buffer "tail"
-/// as we go.
-fn process_block(bank: &Bank, entries: &[Entry]) -> Result<()> {
-    for entry in entries {
-        process_entry(bank, entry)?;
-    }
-
-    Ok(())
-}
-
 #[derive(Debug, PartialEq)]
 pub struct BankForksInfo {
     pub bank_slot: u64,
@@ -167,7 +157,7 @@ pub fn process_blocktree(
                 return Err(BankError::LedgerVerificationFailed);
             }
 
-            process_block(&bank, &entries).map_err(|err| {
+            process_entries(&bank, &entries).map_err(|err| {
                 warn!("Failed to process entries for slot {}: {:?}", slot, err);
                 BankError::LedgerVerificationFailed
             })?;

core/src/crds_gossip.rs

@@ -113,9 +113,9 @@ impl CrdsGossip {
     }
 
     /// time when a request to `from` was initiated
-    /// This is used for weighted random selection durring `new_pull_request`
+    /// This is used for weighted random selection during `new_pull_request`
     /// It's important to use the local nodes request creation time as the weight
-    /// instaad of the response received time otherwise failed nodes will increase their weight.
+    /// instead of the response received time otherwise failed nodes will increase their weight.
     pub fn mark_pull_request_creation_time(&mut self, from: Pubkey, now: u64) {
         self.pull.mark_pull_request_creation_time(from, now)
     }

core/src/local_cluster.rs

@@ -115,10 +115,10 @@ impl LocalCluster {
         }
     }
 
-    pub fn close(&mut self) {
+    fn close(&mut self) {
         self.close_preserve_ledgers();
         for path in &self.ledger_paths {
-            remove_dir_all(path).unwrap();
+            remove_dir_all(path).unwrap_or_else(|_| panic!("Unable to remove {}", path));
         }
     }