leyoda/cosmic-comp
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Only use .cloned() on the element instead of the entire iterator

Browse source
This commit is contained in:
julianbraha 2024-08-29 14:12:26 +01:00 • committed by Victoria Brekenfeld
parent 06d25df4ef
commit 59b0e0e74e
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/shell/layout/tiling/mod.rs
View file

@ -1432,7 +1432,7 @@ impl TilingLayout {
group.remove_window(position); group.remove_window(position);
} else { } else {
trace!("Removing Group"); trace!("Removing Group");
let other_child = tree.children_ids(&id).unwrap().cloned().next().unwrap(); let other_child = tree.children_ids(&id).unwrap().next().cloned().unwrap();
let fork_pos = parent_parent_id.as_ref().and_then(|parent_id| { let fork_pos = parent_parent_id.as_ref().and_then(|parent_id| {
tree.children_ids(parent_id).unwrap().position(|i| i == &id) tree.children_ids(parent_id).unwrap().position(|i| i == &id)
}); });
@ -2984,8 +2984,8 @@ impl TilingLayout {
let child_id = tree let child_id = tree
.children_ids(&node_id) .children_ids(&node_id)
.unwrap() .unwrap()
.cloned()
.next() .next()
.cloned()
.unwrap(); .unwrap();
tree.remove_node(node_id, RemoveBehavior::LiftChildren) tree.remove_node(node_id, RemoveBehavior::LiftChildren)
.unwrap(); .unwrap();