KCL: Remove unnecessary clone in recast (#7845)

adamchalmers · web-flow · commit 60042cb3afb1 · 2025-07-18T17:32:19.000-05:00
Improves perf by 27% to 38% on my mac, 56-88% on Codspeed.
diff --git a/rust/kcl-lib/src/unparser.rs b/rust/kcl-lib/src/unparser.rs
@@ -60,7 +60,7 @@ impl Program {
                 for attr in body_item.get_attrs() {
                     result.push_str(&attr.recast(options, indentation_level));
                 }
-                result.push_str(&match body_item.clone() {
+                result.push_str(&match body_item {
                     BodyItem::ImportStatement(stmt) => stmt.recast(options, indentation_level),
                     BodyItem::ExpressionStatement(expression_statement) => {
                         expression_statement

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ impl Program {`
`60`	`60`	`for attr in body_item.get_attrs() {`
`61`	`61`	`result.push_str(&attr.recast(options, indentation_level));`
`62`	`62`	`}`
`63`		`- result.push_str(&match body_item.clone() {`
	`63`	`+ result.push_str(&match body_item {`
`64`	`64`	`BodyItem::ImportStatement(stmt) => stmt.recast(options, indentation_level),`
`65`	`65`	`BodyItem::ExpressionStatement(expression_statement) => {`
`66`	`66`	`expression_statement`