diff options
author | Sebastian Thiel <sebastian.thiel@icloud.com> | 2021-07-30 08:57:45 +0800 |
---|---|---|
committer | Sebastian Thiel <sebastian.thiel@icloud.com> | 2021-07-30 08:57:45 +0800 |
commit | 4598d64a1150967e48013091e044eae851de62f9 (patch) | |
tree | 9a893edeca5be06eca72d4ad3843c19e3f5ae205 | |
parent | 8222d993a3afd05e17566b6b30d349b6e4080e0d (diff) |
thanks clippy
-rw-r--r-- | src/interactive/app/eventloop.rs | 4 | ||||
-rw-r--r-- | src/interactive/app/tests/mod.rs | 2 | ||||
-rw-r--r-- | src/interactive/app/tests/utils.rs | 74 | ||||
-rw-r--r-- | src/interactive/widgets/main.rs | 2 |
4 files changed, 41 insertions, 41 deletions
diff --git a/src/interactive/app/eventloop.rs b/src/interactive/app/eventloop.rs index 43160dd..f5f89a3 100644 --- a/src/interactive/app/eventloop.rs +++ b/src/interactive/app/eventloop.rs @@ -56,9 +56,9 @@ impl AppState { B: Backend, { let props = MainWindowProps { - traversal: &traversal, + traversal, display, - state: &self, + state: self, }; draw_window(window, props, terminal) } diff --git a/src/interactive/app/tests/mod.rs b/src/interactive/app/tests/mod.rs index ef6efb5..e97ee73 100644 --- a/src/interactive/app/tests/mod.rs +++ b/src/interactive/app/tests/mod.rs @@ -1,4 +1,4 @@ -pub const FIXTURE_PATH: &'static str = "tests/fixtures"; +pub const FIXTURE_PATH: &str = "tests/fixtures"; mod journeys_readonly; mod journeys_with_writes; diff --git a/src/interactive/app/tests/utils.rs b/src/interactive/app/tests/utils.rs index deea2c3..017ade3 100644 --- a/src/interactive/app/tests/utils.rs +++ b/src/interactive/app/tests/utils.rs @@ -30,7 +30,7 @@ pub fn node_by_index(app: &TerminalApp, id: TreeIndex) -> &EntryData { } pub fn node_by_name(app: &TerminalApp, name: impl AsRef<OsStr>) -> &EntryData { - node_by_index(app, index_by_name(&app, name)) + node_by_index(app, index_by_name(app, name)) } pub fn index_by_name_and_size( @@ -135,7 +135,7 @@ fn copy_recursive(src: impl AsRef<Path>, dst: impl AsRef<Path>) -> Result<(), Er impl From<&'static str> for WritableFixture { fn from(fixture_name: &str) -> Self { - const TEMP_TLD_DIRNAME: &'static str = "dua-unit"; + const TEMP_TLD_DIRNAME: &str = "dua-unit"; let src = fixture(fixture_name); let dst = temp_dir().join(TEMP_TLD_DIRNAME); @@ -205,73 +205,75 @@ pub fn initialized_app_and_terminal_from_paths( pub fn initialized_app_and_terminal_from_fixture( fixture_paths: &[&str], ) -> Result<(Terminal<TestBackend>, TerminalApp), Error> { + #[allow(clippy::redundant_closure)] + // doesn't actually work that way due to borrowchk - probably a bug initialized_app_and_terminal_with_closure(fixture_paths, |p| fixture(p)) } pub fn sample_01_tree() -> Tree { - let mut t = Tree::new(); + let mut tree = Tree::new(); { - let mut add_node = make_add_node(&mut t); + let mut add_node = make_add_node(&mut tree); #[cfg(not(windows))] let root_size = 1259070; #[cfg(windows)] let root_size = 1259069; - let r = add_node("", root_size, None); + let rn = add_node("", root_size, None); { - let s = add_node(&fixture_str("sample-01"), root_size, Some(r)); + let sn = add_node(&fixture_str("sample-01"), root_size, Some(rn)); { - add_node(".hidden.666", 666, Some(s)); - add_node("a", 256, Some(s)); - add_node("b.empty", 0, Some(s)); + add_node(".hidden.666", 666, Some(sn)); + add_node("a", 256, Some(sn)); + add_node("b.empty", 0, Some(sn)); #[cfg(not(windows))] - add_node("c.lnk", 1, Some(s)); + add_node("c.lnk", 1, Some(sn)); #[cfg(windows)] - add_node("c.lnk", 0, Some(s)); - let d = add_node("dir", 1258024, Some(s)); + add_node("c.lnk", 0, Some(sn)); + let dn = add_node("dir", 1258024, Some(sn)); { - add_node("1000bytes", 1000, Some(d)); - add_node("dir-a.1mb", 1_000_000, Some(d)); - add_node("dir-a.kb", 1024, Some(d)); - let e = add_node("empty-dir", 0, Some(d)); + add_node("1000bytes", 1000, Some(dn)); + add_node("dir-a.1mb", 1_000_000, Some(dn)); + add_node("dir-a.kb", 1024, Some(dn)); + let en = add_node("empty-dir", 0, Some(dn)); { - add_node(".gitkeep", 0, Some(e)); + add_node(".gitkeep", 0, Some(en)); } - let sub = add_node("sub", 256_000, Some(d)); + let sub = add_node("sub", 256_000, Some(dn)); { add_node("dir-sub-a.256kb", 256_000, Some(sub)); } } - add_node("z123.b", 123, Some(s)); + add_node("z123.b", 123, Some(sn)); } } } - t + tree } pub fn sample_02_tree() -> Tree { - let mut t = Tree::new(); + let mut tree = Tree::new(); { - let mut add_node = make_add_node(&mut t); + let mut add_node = make_add_node(&mut tree); let root_size = 1540; - let r = add_node("", root_size, None); + let rn = add_node("", root_size, None); { - let s = add_node( + let sn = add_node( Path::new(FIXTURE_PATH).join("sample-02").to_str().unwrap(), root_size, - Some(r), + Some(rn), ); { - add_node("a", 256, Some(s)); - add_node("b", 1, Some(s)); - let d = add_node("dir", 1283, Some(s)); + add_node("a", 256, Some(sn)); + add_node("b", 1, Some(sn)); + let dn = add_node("dir", 1283, Some(sn)); { - add_node("c", 257, Some(d)); - add_node("d", 2, Some(d)); - let e = add_node("empty-dir", 0, Some(d)); + add_node("c", 257, Some(dn)); + add_node("d", 2, Some(dn)); + let en = add_node("empty-dir", 0, Some(dn)); { - add_node(".gitkeep", 0, Some(e)); + add_node(".gitkeep", 0, Some(en)); } - let sub = add_node("sub", 1024, Some(d)); + let sub = add_node("sub", 1024, Some(dn)); { add_node("e", 1024, Some(sub)); } @@ -279,12 +281,10 @@ pub fn sample_02_tree() -> Tree { } } } - t + tree } -pub fn make_add_node<'a>( - t: &'a mut Tree, -) -> impl FnMut(&str, u128, Option<NodeIndex>) -> NodeIndex + 'a { +pub fn make_add_node(t: &mut Tree) -> impl FnMut(&str, u128, Option<NodeIndex>) -> NodeIndex + '_ { move |name, size, maybe_from_idx| { let n = t.add_node(EntryData { name: PathBuf::from(name), diff --git a/src/interactive/widgets/main.rs b/src/interactive/widgets/main.rs index cce01c0..83c48f5 100644 --- a/src/interactive/widgets/main.rs +++ b/src/interactive/widgets/main.rs @@ -125,7 +125,7 @@ impl MainWindow { let marked = self.mark_pane.as_ref().map(|p| p.marked()); let props = EntriesProps { - tree: &tree, + tree, root: state.root, display: *display, entries: &state.entries, |