From 9d0bf89447170029cedb456846b5e50a29f163a2 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Tue, 15 Sep 2026 08:17:30 +0000 Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=B9=20[code=20health=20improvement]=20?= =?UTF-8?q?Remove=20unnecessary=20Rc=20clone=20in=20database=20module?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Removed `ctx.resources.clone()` in `std_database_query` and `std_database_execute` in `stdlib/src/database.rs`. Directly borrowing the `Rc` avoids unnecessary reference counting operations, improving code health and performance. All tests have been run and passed. Co-authored-by: Tcode-Motion <188012755+Tcode-Motion@users.noreply.github.com> --- stdlib/src/database.rs | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/stdlib/src/database.rs b/stdlib/src/database.rs index 15987f53..7be0ee38 100644 --- a/stdlib/src/database.rs +++ b/stdlib/src/database.rs @@ -78,8 +78,7 @@ fn std_database_query( let sql = args[1].try_into_string()?; let params_list = get_params_list(&args); - let resources = ctx.resources.clone(); - let resources_borrow = resources.borrow(); + let resources_borrow = ctx.resources.borrow(); let conn = resources_borrow .get::(handle) .ok_or_else(|| { @@ -173,8 +172,7 @@ fn std_database_execute( let sql = args[1].try_into_string()?; let params_list = get_params_list(&args); - let resources = ctx.resources.clone(); - let resources_borrow = resources.borrow(); + let resources_borrow = ctx.resources.borrow(); let conn = resources_borrow .get::(handle) .ok_or_else(|| {