OpenSourceArk/obsidian-export
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Chg: Don't Box FilterFn in WalkOptions

Browse Source 
Previously, `filter_fn` on the `WalkOptions` struct looked like:

    pub filter_fn: Option<Box<&'static FilterFn>>,

This boxing was unneccesary and has been changed to:

    pub filter_fn: Option<&'static FilterFn>,

This will only affect people who use obsidian-export as a library in
other Rust programs, not users of the CLI.

For those library users, they no longer need to supply `FilterFn`
wrapped in a Box.
This commit is contained in:
Nick Groenen 2021-02-12 13:30:06 +01:00
parent 7c7042d1dd
commit f94753c511
No known key found for this signature in database
GPG Key ID: 4F0AD019928AE098
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/walker.rs
View File

@ -28,7 +28,7 @@ pub struct WalkOptions<'a> {
/// it should be included or not. /// it should be included or not.
/// ///
/// This is passed to [`ignore::WalkBuilder::filter_entry`]. /// This is passed to [`ignore::WalkBuilder::filter_entry`].
pub filter_fn: Option<Box<&'static FilterFn>>, pub filter_fn: Option<&'static FilterFn>,
} }
impl<'a> fmt::Debug for WalkOptions<'a> { impl<'a> fmt::Debug for WalkOptions<'a> {