Utf8 checks, using references and avoid allocating

And add docs

src/archive/zip.rs

@@ -10,14 +10,13 @@ use fs_err as fs;
 use walkdir::WalkDir;
 use zip::{self, read::ZipFile, ZipArchive};
 
-use self::utf8::get_invalid_utf8_paths;
 use crate::{
     error::FinalError,
     info,
     list::FileInArchive,
     utils::{
-        cd_into_same_dir_as, concatenate_os_str_list, dir_is_empty, strip_cur_dir, to_utf, user_wants_to_overwrite,
+        cd_into_same_dir_as, concatenate_os_str_list, dir_is_empty, get_invalid_utf8_paths, strip_cur_dir, to_utf,
-        Bytes,
+        user_wants_to_overwrite, Bytes,
     },
     QuestionPolicy,
 };

src/utils/mod.rs

@@ -16,23 +16,15 @@ pub use question::{
 pub use utf8::{get_invalid_utf8_paths, is_invalid_utf8};
 
 mod utf8 {
-    use std::path::{Path, PathBuf};
+    use std::{ffi::OsStr, path::PathBuf};
 
-    pub fn is_invalid_utf8(path: &Path) -> bool {
+    /// Check, without allocating, if os_str can be converted into &str
-        #[cfg(unix)]
+    pub fn is_invalid_utf8(os_str: impl AsRef<OsStr>) -> bool {
-        {
+        os_str.as_ref().to_str().is_none()
-            use std::{os::unix::prelude::OsStrExt, str};
-
-            let bytes = path.as_os_str().as_bytes();
-            str::from_utf8(bytes).is_err()
-        }
-        #[cfg(not(unix))]
-        {
-            path.to_str().is_none()
-        }
     }
 
-    pub fn get_invalid_utf8_paths(paths: &[PathBuf]) -> Vec<PathBuf> {
+    /// Filter out list of paths that are not utf8 valid
-        paths.iter().filter_map(|path| is_invalid_utf8(&path).then(|| path.clone())).collect()
+    pub fn get_invalid_utf8_paths(paths: &[PathBuf]) -> Vec<&PathBuf> {
+        paths.iter().filter_map(|path| is_invalid_utf8(path).then(|| path)).collect()
     }
 }