f Remove KVStoreUnpersister

Author: tnull

src/io/fs_store.rs

@@ -1,7 +1,7 @@
 #[cfg(target_os = "windows")]
 extern crate winapi;
 
-use super::{KVStore, KVStoreUnpersister, TransactionalWrite};
+use super::{KVStore, TransactionalWrite};
 
 use std::fs;
 use std::io::{BufReader, BufWriter, Read, Write};
@@ -90,7 +90,7 @@ impl KVStore for FilesystemStore {
 		}
 
 		if dest_file.is_file() {
-			return Err(std::io::Error::new(std::io::ErrorKind::Other, "Unpersisting key failed"));
+			return Err(std::io::Error::new(std::io::ErrorKind::Other, "Removing key failed"));
 		}
 
 		Ok(true)
@@ -256,20 +256,6 @@ impl KVStorePersister for FilesystemStore {
 	}
 }
 
-impl KVStoreUnpersister for FilesystemStore {
-	fn unpersist(&self, key: &str) -> std::io::Result<bool> {
-		let msg = format!("Could not retrieve file for key {}.", key);
-		let dest_file = PathBuf::from_str(key)
-			.map_err(|_| lightning::io::Error::new(lightning::io::ErrorKind::InvalidInput, msg))?;
-		let msg = format!("Could not retrieve parent directory of {}.", dest_file.display());
-		let parent_directory = dest_file
-			.parent()
-			.ok_or(lightning::io::Error::new(lightning::io::ErrorKind::InvalidInput, msg))?;
-		let namespace = parent_directory.display().to_string();
-		self.remove(&namespace, key)
-	}
-}
-
 #[cfg(test)]
 mod tests {
 	use super::*;

src/io/mod.rs

@@ -95,10 +95,3 @@ pub trait TransactionalWrite: Write {
 //		Ok(writer.commit()?)
 //	}
 //}
-
-/// Provides an interface that allows a previously persisted key to be unpersisted.
-pub trait KVStoreUnpersister {
-	/// Unpersist (i.e., remove) the writeable previously persisted under the provided key.
-	/// Returns `true` if the key was present, and `false` otherwise.
-	fn unpersist(&self, key: &str) -> std::io::Result<bool>;
-}