fix: small typo+ for non-major supported systems

Author: alexcu2718

benches/dirent_bench.rs

@@ -146,9 +146,7 @@ fn bench_strlen(c: &mut Criterion) {
                 let mut total = 0;
                 for entry in &all_entries {
                     total += unsafe {
-                        black_box(libc::strlen(black_box(
-                            access_dirent!(entry, d_name),
-                        )))
+                        black_box(libc::strlen(black_box(access_dirent!(entry, d_name))))
                     };
                 }
                 black_box(total) //make sure compiler does not optimise this away

src/direntry.rs

@@ -885,7 +885,12 @@ impl DirEntry {
      Returns `DirEntryError::IOError` if the stat operation fails
     */
     pub fn get_lstatat(&self, fd: &FileDes) -> Result<stat> {
-        stat_syscall!(fstatat, fd.0, self.file_name_cstr().as_ptr(), AT_SYMLINK_NOFOLLOW)
+        stat_syscall!(
+            fstatat,
+            fd.0,
+            self.file_name_cstr().as_ptr(),
+            AT_SYMLINK_NOFOLLOW
+        )
     }
 
     #[inline]
@@ -961,7 +966,12 @@ impl DirEntry {
     *
     */
     pub fn get_statat(&self, fd: &FileDes) -> Result<stat> {
-        stat_syscall!(fstatat, fd.0, self.file_name_cstr().as_ptr(), AT_SYMLINK_FOLLOW)
+        stat_syscall!(
+            fstatat,
+            fd.0,
+            self.file_name_cstr().as_ptr(),
+            AT_SYMLINK_FOLLOW
+        )
     }
 
     #[inline]

src/iter.rs

@@ -5,7 +5,7 @@ use crate::{dirent64, readdir64};
 use core::cell::Cell;
 use core::ffi::CStr;
 use core::ptr::NonNull;
- use libc::{fstatat,AT_SYMLINK_NOFOLLOW,DIR};
+use libc::{AT_SYMLINK_NOFOLLOW, DIR, fstatat};
 /**
  POSIX-compliant directory iterator using libc's readdir
 
@@ -381,15 +381,12 @@ pub trait DirentConstructor {
     /// Returns the file descriptor for the current directory being read
     fn file_descriptor(&self) -> &FileDes;
 
-  
-
     #[inline]
-    #[allow(clippy::wildcard_enum_match_arm,reason="exhaustive")]
+    #[allow(clippy::wildcard_enum_match_arm, reason = "exhaustive")]
     #[allow(clippy::multiple_unsafe_ops_per_block)]
     /// Constructs a `DirEntry` from a raw directory entry pointer
     #[allow(unused_unsafe)] //lazy fix for illumos/solaris (where we dont actually dereference the pointer, just return unknown TODO-MAKE MORE ELEGANT)
     unsafe fn construct_entry(&mut self, drnt: *const dirent64) -> DirEntry {
-       
         let base_len = self.file_index();
         debug_assert!(!drnt.is_null(), "drnt should never be null!");
         // SAFETY: The `drnt` must not be null (checked before using)
@@ -398,12 +395,18 @@ pub trait DirentConstructor {
         let inode = unsafe { access_dirent!(drnt, d_ino) };
 
         // SAFETY: The `drnt` must not be null(by precondition)
-        let full_path:&CStr = unsafe { self.construct_path(drnt) };
+        let full_path: &CStr = unsafe { self.construct_path(drnt) };
         let path: Box<CStr> = full_path.into();
 
-        let file_type:FileType = match FileType::from_dtype(dtype){
-            FileType::Unknown=>stat_syscall!(fstatat,self.file_descriptor().0,access_dirent!(drnt,d_name),AT_SYMLINK_NOFOLLOW,DTYPE),
-            not_unknown=>not_unknown //if not unknown, skip the syscall (THIS IS A MASSIVE PERF WIN)
+        let file_type: FileType = match FileType::from_dtype(dtype) {
+            FileType::Unknown => stat_syscall!(
+                fstatat,
+                self.file_descriptor().0,
+                access_dirent!(drnt, d_name),
+                AT_SYMLINK_NOFOLLOW,
+                DTYPE
+            ),
+            not_unknown => not_unknown, //if not unknown, skip the syscall (THIS IS A MASSIVE PERF WIN)
         };
 
         DirEntry {

src/lib.rs

@@ -334,7 +334,7 @@ impl Finder {
         use_colours: bool,
         result_count: Option<usize>,
         sort: bool,
-        print_errors:bool
+        print_errors: bool,
     ) -> core::result::Result<(), SearchConfigError> {
         let errors = self.errors.clone();
         let iter = self.traverse()?;

src/macros.rs

@@ -249,7 +249,7 @@ macro_rules! skip_dot_or_dot_dot_entries {
                     // Only check d_type for potential "." or ".." entries
                     match access_dirent!($entry, d_type) {
                         libc::DT_DIR | libc::DT_UNKNOWN => {
-                            let name_ptr:*const u8 = access_dirent!($entry, d_name);
+                            let name_ptr: *const u8 = access_dirent!($entry, d_name);
                             // Combined check using pattern
                             match (namelen, *name_ptr.add(0), *name_ptr.add(1)) {
                                 (1, b'.', _) => $action,    // "." - length 1, first char '.'
@@ -277,7 +277,7 @@ macro_rules! skip_dot_or_dot_dot_entries {
                     libc::DT_DIR | libc::DT_UNKNOWN => {
                         // The value for 24 is checked in util.
                         if access_dirent!($entry, d_reclen) == MINIMUM_DIRENT_SIZE {
-                            let name_ptr:*const u8 = access_dirent!($entry, d_name);
+                            let name_ptr: *const u8 = access_dirent!($entry, d_name);
                             match (*name_ptr.add(0), *name_ptr.add(1), *name_ptr.add(2)) {
                                 (b'.', 0, _) | (b'.', b'.', 0) => $action, //similar to above
                                 _ => (),
@@ -303,7 +303,7 @@ macro_rules! skip_dot_or_dot_dot_entries {
                 // Fallback for other systems: check d_type first
                 match access_dirent!($entry, d_type) {
                     libc::DT_DIR | libc::DT_UNKNOWN => {
-                        let name_ptr:*const u8 = access_dirent!($entry, d_name);
+                        let name_ptr: *const u8 = access_dirent!($entry, d_name);
                         match (*name_ptr.add(0), *name_ptr.add(1), *name_ptr.add(2)) {
                             (b'.', 0, _) | (b'.', b'.', 0) => $action,
                             _ => (),

src/main.rs

@@ -281,7 +281,7 @@ fn main() -> Result<(), SearchConfigError> {
         .thread_count(args.thread_num.unwrap_or(THREAD_COUNT))
         .build()?;
 
-    let _ = finder.print_results(args.no_colour, args.top_n, args.sort,args.show_errors);
+    let _ = finder.print_results(args.no_colour, args.top_n, args.sort, args.show_errors);
 
     Ok(())
 }

src/test.rs

@@ -11,6 +11,7 @@ mod tests {
     use filetime::{FileTime, set_file_times};
     use std::env::temp_dir;
     use std::ffi::OsStr;
+    use std::ffi::OsString;
     use std::fs;
     use std::fs::File;
     use std::io::Write;
@@ -19,7 +20,6 @@ mod tests {
     use std::os::unix::fs::symlink;
     use std::path::{Path, PathBuf};
     use std::sync::Arc;
-    use std::{ffi::OsString};
 
     use std::time::{Duration, SystemTime, UNIX_EPOCH};