tsort: use Option::inspect over map

jfinkels · jfinkels · commit 62879244cbfa · 2025-01-11T11:58:51.000-05:00
diff --git a/src/uu/tsort/src/tsort.rs b/src/uu/tsort/src/tsort.rs
@@ -105,9 +105,8 @@ fn remove<T>(vec: &mut Vec<T>, x: T) -> Option<usize>
 where
     T: PartialEq,
 {
-        vec.iter().position(|item| *item == x).map(|i| {
-        vec.remove(i);
-        i
+    vec.iter().position(|item| *item == x).inspect(|i| {
+        vec.remove(*i);
     })
 }
 

Original file line number	Diff line number	Diff line change
`@@ -105,9 +105,8 @@ fn remove<T>(vec: &mut Vec<T>, x: T) -> Option<usize>`
`105`	`105`	`where`
`106`	`106`	`T: PartialEq,`
`107`	`107`	`{`
`108`		`- vec.iter().position(\|item\| *item == x).map(\|i\| {`
`109`		`- vec.remove(i);`
`110`		`- i`
	`108`	`+ vec.iter().position(\|item\| *item == x).inspect(\|i\| {`
	`109`	`+ vec.remove(*i);`
`111`	`110`	`})`
`112`	`111`	`}`
`113`	`112`