Merge pull request #20122 from chenyukang/yukang-fix-remove-parens

ChayimFriedman2 · web-flow · commit 6df12139bcca · 2025-06-29T15:14:31.000Z
Remove unnecessary parens in closure
diff --git a/crates/ide-completion/src/completions/fn_param.rs b/crates/ide-completion/src/completions/fn_param.rs
@@ -195,5 +195,5 @@ fn comma_wrapper(ctx: &CompletionContext<'_>) -> Option<(impl Fn(&str) -> String
         matches!(prev_token_kind, SyntaxKind::COMMA | SyntaxKind::L_PAREN | SyntaxKind::PIPE);
     let leading = if has_leading_comma { "" } else { ", " };
 
-    Some((move |label: &_| (format!("{leading}{label}{trailing}")), param.text_range()))
+    Some((move |label: &_| format!("{leading}{label}{trailing}"), param.text_range()))
 }

Original file line number	Diff line number	Diff line change
`@@ -195,5 +195,5 @@ fn comma_wrapper(ctx: &CompletionContext<'_>) -> Option<(impl Fn(&str) -> String`
`195`	`195`	`matches!(prev_token_kind, SyntaxKind::COMMA \| SyntaxKind::L_PAREN \| SyntaxKind::PIPE);`
`196`	`196`	`let leading = if has_leading_comma { "" } else { ", " };`
`197`	`197`
`198`		`- Some((move \|label: &_\| (format!("{leading}{label}{trailing}")), param.text_range()))`
	`198`	`+ Some((move \|label: &_\| format!("{leading}{label}{trailing}"), param.text_range()))`
`199`	`199`	`}`