Remove a redundant into_iter

ndmitchell · facebook-github-bot · commit b5f679f7a615 · 2023-07-05T07:08:22.000-07:00
Summary: Clippy says it is redundant.

Reviewed By: JakobDegen

Differential Revision: D47226537

fbshipit-source-id: 91aeea99b38218e27402a0496bc0c531210332c8
diff --git a/starlark/src/docs/mod.rs b/starlark/src/docs/mod.rs
@@ -832,7 +832,7 @@ pub fn render_docs_as_code(docs: &[Doc]) -> String {
         .partition(|d| matches!(d.item, DocItem::Module(_)));
     modules
         .into_iter()
-        .chain(non_modules.into_iter())
+        .chain(non_modules)
         .map(|d| d.render_as_code())
         .join("\n\n")
 }

Original file line number	Diff line number	Diff line change
`@@ -832,7 +832,7 @@ pub fn render_docs_as_code(docs: &[Doc]) -> String {`
`832`	`832`	`.partition(\|d\| matches!(d.item, DocItem::Module(_)));`
`833`	`833`	`modules`
`834`	`834`	`.into_iter()`
`835`		`- .chain(non_modules.into_iter())`
	`835`	`+ .chain(non_modules)`
`836`	`836`	`.map(\|d\| d.render_as_code())`
`837`	`837`	`.join("\n\n")`
`838`	`838`	`}`