Add unified Filter trait for consistent filtering interface

src/filter/mod.rs

@@ -9,3 +9,9 @@ mod time;
 
 #[cfg(unix)]
 mod owner;
+
+pub trait Filter {
+    type Item;
+
+    fn matches(&self, item: &Self::Item) -> bool;
+}

src/filter/size.rs

@@ -3,6 +3,8 @@ use std::sync::OnceLock;
 use anyhow::anyhow;
 use regex::Regex;
 
+use super::Filter;
+
 static SIZE_CAPTURES: OnceLock<Regex> = OnceLock::new();
 
 #[derive(Clone, Copy, Debug, PartialEq, Eq)]
@@ -74,6 +76,15 @@ impl SizeFilter {
     }
 }
 
+impl Filter for SizeFilter {
+    type Item = u64;
+
+    #[inline(always)]
+    fn matches(&self, item: &Self::Item) -> bool {
+        self.is_within(*item)
+    }
+}
+
 #[cfg(test)]
 mod tests {
     use super::*;

src/filter/time.rs

@@ -2,6 +2,8 @@ use jiff::{Span, Timestamp, Zoned, civil::DateTime, tz::TimeZone};
 
 use std::time::{Duration, SystemTime, UNIX_EPOCH};
 
+use super::Filter;
+
 /// Filter based on time ranges.
 #[derive(Debug, PartialEq, Eq)]
 pub enum TimeFilter {
@@ -62,6 +64,15 @@ impl TimeFilter {
     }
 }
 
+impl Filter for TimeFilter {
+    type Item = SystemTime;
+
+    #[inline(always)]
+    fn matches(&self, item: &Self::Item) -> bool {
+        self.applies_to(item)
+    }
+}
+
 #[cfg(test)]
 mod tests {
     use super::*;

src/walk.rs

@@ -21,6 +21,7 @@ use crate::error::print_error;
 use crate::exec;
 use crate::exit_codes::{ExitCode, merge_exitcodes};
 use crate::filesystem;
+use crate::filter::Filter;
 use crate::output;
 
 /// The receiver thread can either be buffering results or directly streaming to the console.
@@ -568,7 +569,7 @@ impl WorkerState {
                             if config
                                 .size_constraints
                                 .iter()
-                                .any(|sc| !sc.is_within(file_size))
+                                .any(|sc| !sc.matches(&file_size))
                             {
                                 return WalkState::Continue;
                             }
@@ -589,7 +590,7 @@ impl WorkerState {
                         matched = config
                             .time_constraints
                             .iter()
-                            .all(|tf| tf.applies_to(&modified));
+                            .all(|tf| tf.matches(&modified));
                     }
                     if !matched {
                         return WalkState::Continue;