From 3ea784aff760e93c626253d28f3e42d3d600b18c Mon Sep 17 00:00:00 2001
From: Rob Walker <rob@solana.com>
Date: Wed, 1 Aug 2018 11:02:20 -0700
Subject: [PATCH] clippy fixups

---
 src/ledger.rs | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/ledger.rs b/src/ledger.rs
index d089f8a6c1..c6b1366edc 100644
--- a/src/ledger.rs
+++ b/src/ledger.rs
@@ -28,7 +28,7 @@ const SIZEOF_U64: u64 = size_of::<u64>() as u64;
 
 impl LedgerWindow {
     // opens a Ledger in directory, provides "infinite" window
-    pub fn new(directory: String) -> io::Result<Self> {
+    pub fn new(directory: &str) -> io::Result<Self> {
         let directory = Path::new(&directory);
 
         let index = File::open(directory.join("index"))?;
@@ -71,7 +71,7 @@ pub struct LedgerWriter {
 
 impl LedgerWriter {
     // opens or creates a LedgerWriter in directory
-    pub fn new(directory: String) -> io::Result<Self> {
+    pub fn new(directory: &str) -> io::Result<Self> {
         let directory = Path::new(&directory);
 
         create_dir_all(directory)?;
@@ -143,7 +143,7 @@ impl Iterator for LedgerReader {
 }
 
 /// Return an iterator for all the entries in the given file.
-pub fn read_ledger(directory: String) -> io::Result<impl Iterator<Item = io::Result<Entry>>> {
+pub fn read_ledger(directory: &str) -> io::Result<impl Iterator<Item = io::Result<Entry>>> {
     let directory = Path::new(&directory);
 
     let index = File::open(directory.join("index"))?;
@@ -457,6 +457,7 @@ mod tests {
             let read_entry = window.get_entry(i as u64).unwrap();
             assert_eq!(*entry, read_entry);
         }
+        assert!(window.get_entry(100).is_err());
 
         std::fs::remove_file(Path::new(&ledger_path).join("data")).unwrap();
         // empty data file should fall over