refactor: how FsEntry structs are created

Now from a Metadata struct, instead of a bunch of field values. The
justification for this is that callers shouldn't have to unpack a
Metadata, especially since it'll be different for each operating
system in the future. Keep all that in one place instead.

1 files changed, 2 insertions, 6 deletions

diff --git a/src/fsiter.rs b/src/fsiter.rs
index 929c81e..3c08179 100644
--- a/src/fsiter.rs
+++ b/src/fsiter.rs
@@ -28,10 +28,6 @@ impl Iterator for FsIterator {
 
 fn new_entry(e: &walkdir::DirEntry) -> anyhow::Result<FilesystemEntry> {
     let meta = e.metadata()?;
-    let kind = if meta.is_dir() {
-        FilesystemEntry::directory(e.path())
-    } else {
-        FilesystemEntry::regular(e.path(), meta.len())
-    };
-    Ok(kind)
+    let entry = FilesystemEntry::from_metadata(e.path(), &meta);
+    Ok(entry)
 }