about summary refs log tree commit diff
path: root/tvix/store
diff options
context:
space:
mode:
authorFlorian Klink <flokli@flokli.de>2023-05-25T05·29+0300
committerclbot <clbot@tvl.fyi>2023-05-25T11·11+0000
commitfcfbcf9cfa52e221056ea52f7b70e06ca0a36eba (patch)
treeb12b1ab0c8b764ea2e2e42705c6c23d83b151ef4 /tvix/store
parentca06f7061ca298244fcd61d9c957b3590f40f59f (diff)
refactor(tvix/store): drop mut self borrow in ingest_path r/6200
With traverse_to not requiring a &mut anymore, we can drop the &mut self
in all these function signatures.

Change-Id: I22105376b625cb281c39e92d3206df8a6ce97a88
Reviewed-on: https://cl.tvl.fyi/c/depot/+/8629
Tested-by: BuildkiteCI
Reviewed-by: tazjin <tazjin@tvl.su>
Autosubmit: flokli <flokli@flokli.de>
Diffstat (limited to 'tvix/store')
-rw-r--r--tvix/store/src/import.rs6
-rw-r--r--tvix/store/src/store_io.rs7
2 files changed, 6 insertions, 7 deletions
diff --git a/tvix/store/src/import.rs b/tvix/store/src/import.rs
index ade700a6c283..206e5eaba975 100644
--- a/tvix/store/src/import.rs
+++ b/tvix/store/src/import.rs
@@ -56,7 +56,7 @@ impl From<super::Error> for Error {
 // It assumes the caller adds returned nodes to the directories it assembles.
 #[instrument(skip_all, fields(entry.file_type=?&entry.file_type(),entry.path=?entry.path()))]
 fn process_entry<BS: BlobService, DP: DirectoryPutter>(
-    blob_service: &mut BS,
+    blob_service: &BS,
     directory_putter: &mut DP,
     entry: &walkdir::DirEntry,
     maybe_directory: Option<proto::Directory>,
@@ -145,8 +145,8 @@ fn process_entry<BS: BlobService, DP: DirectoryPutter>(
 /// naming scheme.
 #[instrument(skip(blob_service, directory_service), fields(path=?p))]
 pub fn ingest_path<BS: BlobService, DS: DirectoryService, P: AsRef<Path> + Debug>(
-    blob_service: &mut BS,
-    directory_service: &mut DS,
+    blob_service: &BS,
+    directory_service: &DS,
     p: P,
 ) -> Result<proto::node::Node, Error> {
     // Probe if the path points to a symlink. If it does, we process it manually,
diff --git a/tvix/store/src/store_io.rs b/tvix/store/src/store_io.rs
index e4c45f9e5952..a7467362d334 100644
--- a/tvix/store/src/store_io.rs
+++ b/tvix/store/src/store_io.rs
@@ -96,7 +96,7 @@ impl<BS: BlobService, DS: DirectoryService, PS: PathInfoService, NCS: NARCalcula
             }
         };
 
-        directoryservice::traverse_to(&mut self.directory_service, root_node, sub_path)
+        directoryservice::traverse_to(&self.directory_service, root_node, sub_path)
     }
 
     /// Imports a given path on the filesystem into the store, and returns the
@@ -110,9 +110,8 @@ impl<BS: BlobService, DS: DirectoryService, PS: PathInfoService, NCS: NARCalcula
         path: &std::path::Path,
     ) -> Result<crate::proto::PathInfo, io::Error> {
         // Call [import::ingest_path], which will walk over the given path and return a root_node.
-        let root_node =
-            import::ingest_path(&mut self.blob_service, &mut self.directory_service, path)
-                .expect("error during import_path");
+        let root_node = import::ingest_path(&self.blob_service, &self.directory_service, path)
+            .expect("error during import_path");
 
         // Render the NAR
         let (nar_size, nar_sha256) = self