diff options
Diffstat (limited to 'tvix')
-rw-r--r-- | tvix/cli/src/nix_compat.rs | 6 | ||||
-rw-r--r-- | tvix/cli/src/tvix_io.rs | 6 | ||||
-rw-r--r-- | tvix/eval/src/io.rs | 18 | ||||
-rw-r--r-- | tvix/eval/src/nix_search_path.rs | 2 | ||||
-rw-r--r-- | tvix/eval/src/vm/generators.rs | 6 |
5 files changed, 19 insertions, 19 deletions
diff --git a/tvix/cli/src/nix_compat.rs b/tvix/cli/src/nix_compat.rs index ce352f0a964c..b322886c9e20 100644 --- a/tvix/cli/src/nix_compat.rs +++ b/tvix/cli/src/nix_compat.rs @@ -47,7 +47,7 @@ impl EvalIO for NixCompatIO { } // Pass the rest of the functions through to `Self::underlying` - fn path_exists(&mut self, path: PathBuf) -> Result<bool, io::Error> { + fn path_exists(&mut self, path: &Path) -> Result<bool, io::Error> { if path.starts_with("/__corepkgs__") { return Ok(true); } @@ -55,7 +55,7 @@ impl EvalIO for NixCompatIO { self.underlying.path_exists(path) } - fn read_to_string(&mut self, path: PathBuf) -> Result<String, io::Error> { + fn read_to_string(&mut self, path: &Path) -> Result<String, io::Error> { // Bundled version of corepkgs/fetchurl.nix. This workaround // is similar to what cppnix does for passing the path // through. @@ -69,7 +69,7 @@ impl EvalIO for NixCompatIO { self.underlying.read_to_string(path) } - fn read_dir(&mut self, path: PathBuf) -> Result<Vec<(SmolStr, FileType)>, io::Error> { + fn read_dir(&mut self, path: &Path) -> Result<Vec<(SmolStr, FileType)>, io::Error> { self.underlying.read_dir(path) } } diff --git a/tvix/cli/src/tvix_io.rs b/tvix/cli/src/tvix_io.rs index 5a3f5b0e7a1a..cc3278605572 100644 --- a/tvix/cli/src/tvix_io.rs +++ b/tvix/cli/src/tvix_io.rs @@ -48,7 +48,7 @@ impl<T: EvalIO> EvalIO for TvixIO<T> { Ok(imported_path) } - fn path_exists(&mut self, path: PathBuf) -> Result<bool, io::Error> { + fn path_exists(&mut self, path: &Path) -> Result<bool, io::Error> { if path.starts_with("/__corepkgs__") { return Ok(true); } @@ -56,7 +56,7 @@ impl<T: EvalIO> EvalIO for TvixIO<T> { self.actual.path_exists(path) } - fn read_to_string(&mut self, path: PathBuf) -> Result<String, io::Error> { + fn read_to_string(&mut self, path: &Path) -> Result<String, io::Error> { // Bundled version of corepkgs/fetchurl.nix. The counterpart // of this happens in `main`, where the `nix_path` of the // evaluation has `nix=/__corepkgs__` added to it. @@ -73,7 +73,7 @@ impl<T: EvalIO> EvalIO for TvixIO<T> { self.actual.read_to_string(path) } - fn read_dir(&mut self, path: PathBuf) -> Result<Vec<(SmolStr, FileType)>, io::Error> { + fn read_dir(&mut self, path: &Path) -> Result<Vec<(SmolStr, FileType)>, io::Error> { self.actual.read_dir(path) } } diff --git a/tvix/eval/src/io.rs b/tvix/eval/src/io.rs index 3299c92d1fdb..f14da723db90 100644 --- a/tvix/eval/src/io.rs +++ b/tvix/eval/src/io.rs @@ -33,14 +33,14 @@ pub enum FileType { /// Defines how filesystem interaction occurs inside of tvix-eval. pub trait EvalIO { /// Verify whether the file at the specified path exists. - fn path_exists(&mut self, path: PathBuf) -> Result<bool, io::Error>; + fn path_exists(&mut self, path: &Path) -> Result<bool, io::Error>; /// Read the file at the specified path to a string. - fn read_to_string(&mut self, path: PathBuf) -> Result<String, io::Error>; + fn read_to_string(&mut self, path: &Path) -> Result<String, io::Error>; /// Read the directory at the specified path and return the names /// of its entries associated with their [`FileType`]. - fn read_dir(&mut self, path: PathBuf) -> Result<Vec<(SmolStr, FileType)>, io::Error>; + fn read_dir(&mut self, path: &Path) -> Result<Vec<(SmolStr, FileType)>, io::Error>; /// Import the given path. What this means depends on the /// implementation, for example for a `std::io`-based @@ -65,15 +65,15 @@ pub struct StdIO; #[cfg(feature = "impure")] impl EvalIO for StdIO { - fn path_exists(&mut self, path: PathBuf) -> Result<bool, io::Error> { + fn path_exists(&mut self, path: &Path) -> Result<bool, io::Error> { path.try_exists() } - fn read_to_string(&mut self, path: PathBuf) -> Result<String, io::Error> { + fn read_to_string(&mut self, path: &Path) -> Result<String, io::Error> { std::fs::read_to_string(&path) } - fn read_dir(&mut self, path: PathBuf) -> Result<Vec<(SmolStr, FileType)>, io::Error> { + fn read_dir(&mut self, path: &Path) -> Result<Vec<(SmolStr, FileType)>, io::Error> { let mut result = vec![]; for entry in path.read_dir()? { @@ -108,21 +108,21 @@ impl EvalIO for StdIO { pub struct DummyIO; impl EvalIO for DummyIO { - fn path_exists(&mut self, _: PathBuf) -> Result<bool, io::Error> { + fn path_exists(&mut self, _: &Path) -> Result<bool, io::Error> { Err(io::Error::new( io::ErrorKind::Unsupported, "I/O methods are not implemented in DummyIO", )) } - fn read_to_string(&mut self, _: PathBuf) -> Result<String, io::Error> { + fn read_to_string(&mut self, _: &Path) -> Result<String, io::Error> { Err(io::Error::new( io::ErrorKind::Unsupported, "I/O methods are not implemented in DummyIO", )) } - fn read_dir(&mut self, _: PathBuf) -> Result<Vec<(SmolStr, FileType)>, io::Error> { + fn read_dir(&mut self, _: &Path) -> Result<Vec<(SmolStr, FileType)>, io::Error> { Err(io::Error::new( io::ErrorKind::Unsupported, "I/O methods are not implemented in DummyIO", diff --git a/tvix/eval/src/nix_search_path.rs b/tvix/eval/src/nix_search_path.rs index 8e23c2d9c24b..79c19752f6c1 100644 --- a/tvix/eval/src/nix_search_path.rs +++ b/tvix/eval/src/nix_search_path.rs @@ -85,7 +85,7 @@ impl NixSearchPathEntry { } }; - if io.path_exists(path.clone()).map_err(|e| ErrorKind::IO { + if io.path_exists(&path).map_err(|e| ErrorKind::IO { path: Some(path.clone()), error: e.into(), })? { diff --git a/tvix/eval/src/vm/generators.rs b/tvix/eval/src/vm/generators.rs index 0e3123ae3758..f86683dff542 100644 --- a/tvix/eval/src/vm/generators.rs +++ b/tvix/eval/src/vm/generators.rs @@ -421,7 +421,7 @@ impl<'o> VM<'o> { VMRequest::ReadToString(path) => { let content = self .io_handle - .read_to_string(path.clone()) + .read_to_string(&path) .map_err(|e| ErrorKind::IO { path: Some(path), error: e.into(), @@ -434,7 +434,7 @@ impl<'o> VM<'o> { VMRequest::PathExists(path) => { let exists = self .io_handle - .path_exists(path.clone()) + .path_exists(&path) .map_err(|e| ErrorKind::IO { path: Some(path), error: e.into(), @@ -448,7 +448,7 @@ impl<'o> VM<'o> { VMRequest::ReadDir(path) => { let dir = self .io_handle - .read_dir(path.clone()) + .read_dir(&path) .map_err(|e| ErrorKind::IO { path: Some(path), error: e.into(), |