From d1960a8b7488dffe161080714447120fa6f85db1 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sat, 18 Jul 2026 00:48:13 +0900 Subject: refactor(curl-downloader): drop unnecessary Rc wrapping around handles CurlDownloader::auth_helper was never cloned out to another owner, so Rc> only needed the RefCell for interior mutability (all methods take &self). Likewise HttpDownloader::dispatch cloned self.rfs into a local binding it only ever used synchronously (copy/get_contents don't await), so the clone bought nothing. Co-Authored-By: Claude Sonnet 5 --- crates/shirabe/src/util/http_downloader.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'crates/shirabe/src/util/http_downloader.rs') diff --git a/crates/shirabe/src/util/http_downloader.rs b/crates/shirabe/src/util/http_downloader.rs index c9e1e4a6..98678ad2 100644 --- a/crates/shirabe/src/util/http_downloader.rs +++ b/crates/shirabe/src/util/http_downloader.rs @@ -330,7 +330,7 @@ impl HttpDownloader { }; } - let rfs = self.rfs.as_ref().unwrap().clone(); + let rfs = self.rfs.as_ref().unwrap(); if let Some(copy_to) = copy_to { let (_, headers) = rfs.borrow_mut() -- cgit v1.3.1