Improved Cache Command Output

cli/Cargo.toml

@@ -7,6 +7,7 @@ license.workspace = true
 authors = ["Redfire <[email protected]>"]
 
 [dependencies]
+humansize = "2.1.3"
 rustyline = "12.0.0"
 rustyline-derive = "0.9.0"
 

cli/src/commands/cache.rs

@@ -8,13 +8,14 @@ use std::fs::{metadata, read_dir};
 use std::io;
 use std::path::Path;
 
+use humansize::{BINARY, SizeFormatter};
 use runtime::cache::Cache;
 
 pub(crate) fn cache_statistics() {
 	if let Some(cache) = Cache::new() {
 		println!("Location: {}", cache.dir().display());
 		match cache_size(cache.dir()) {
-			Ok(size) => println!("Size: {}", format_size(size)),
+			Ok(size) => println!("Size: {}", SizeFormatter::new(size, BINARY)),
 			Err(err) => eprintln!("Error while Calculating Size: {}", err),
 		}
 	} else {
@@ -34,21 +35,3 @@ fn cache_size(folder: &Path) -> io::Result<u64> {
 	}
 	Ok(size)
 }
-
-const PREFIXES: [&str; 6] = ["", "Ki", "Mi", "Gi", "Ti", "Pi"];
-
-fn format_size(size: u64) -> String {
-	if size >= 1024 {
-		let index: u32 = f64::log(size as f64, 1024.0).floor() as u32;
-		let s1 = size / 1024_u64.pow(index);
-		let s2 = (size - s1 * 1024_u64.pow(index)) / 1024_u64.pow(index - 1);
-
-		if s2 != 0 {
-			format!("{} {}B, {} {}B", s1, PREFIXES[index as usize], s2, PREFIXES[index as usize - 1])
-		} else {
-			format!("{} {}B", s1, PREFIXES[index as usize])
-		}
-	} else {
-		format!("{} B", size)
-	}
-}

modules/src/path/path.rs

@@ -26,10 +26,7 @@ const DELIMITER: &str = ":\0";
 #[js_fn]
 fn join(#[ion(varargs)] segments: Vec<String>) -> String {
 	let mut path = PathBuf::new();
-	for segment in segments {
-		path.push(segment);
-	}
-
+	path.extend(segments);
 	String::from(path.to_str().unwrap())
 }
 

runtime/src/cache/cache.rs

@@ -28,10 +28,10 @@ pub struct Cache {
 
 impl Cache {
 	pub fn new() -> Option<Cache> {
-		home_dir().map(|path| {
-			let dir = path.join(".spiderfire/cache");
-			let _ = create_dir_all(&dir);
-			Cache { dir }
+		home_dir().map(|mut path| {
+			path.extend([".spiderfire", "cache"]);
+			let _ = create_dir_all(&path);
+			Cache { dir: path }
 		})
 	}