diff options
| author | Florian Klink <flokli@flokli.de> | 2023-05-25T05·29+0300 |
|---|---|---|
| committer | clbot <clbot@tvl.fyi> | 2023-05-25T11·11+0000 |
| commit | fcfbcf9cfa52e221056ea52f7b70e06ca0a36eba (patch) | |
| tree | b12b1ab0c8b764ea2e2e42705c6c23d83b151ef4 | |
| parent | ca06f7061ca298244fcd61d9c957b3590f40f59f (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>
| -rw-r--r-- | tvix/store/src/import.rs | 6 | ||||
| -rw-r--r-- | tvix/store/src/store_io.rs | 7 |
2 files changed, 6 insertions, 7 deletions
diff --git a/tvix/store/src/import.rs b/tvix/store/src/import.rs index ade700a6c2..206e5eaba9 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 e4c45f9e59..a7467362d3 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 |