diff rust/hg-cpython/src/vfs.rs @ 52294:645d247d4c75

rust-vfs: rename `open` to `open_write` and `open_read` to `open` `open` being read *and* write is surprising because it differs from the Rust stdlib where `std::fs::File::open` is read-only by default. More importantly, writing is more dangerous than reading, so let's make it more explicit.
author Rapha?l Gom?s <rgomes@octobus.net>
date Tue, 29 Oct 2024 12:03:55 +0100
parents 8d35941689af
children
line wrap: on
line diff
--- a/rust/hg-cpython/src/vfs.rs	Tue Oct 29 11:41:27 2024 +0100
+++ b/rust/hg-cpython/src/vfs.rs	Tue Oct 29 12:03:55 2024 +0100
@@ -142,11 +142,12 @@
 
 impl Vfs for PyVfs {
     fn open(&self, filename: &Path) -> Result<VfsFile, HgError> {
-        self.inner_open(filename, false, false, false, true)
+        self.inner_open(filename, false, false, false, false)
             .map(|(f, _)| VfsFile::normal(f, filename.to_owned()))
     }
-    fn open_read(&self, filename: &Path) -> Result<VfsFile, HgError> {
-        self.inner_open(filename, false, false, false, false)
+
+    fn open_write(&self, filename: &Path) -> Result<VfsFile, HgError> {
+        self.inner_open(filename, false, false, false, true)
             .map(|(f, _)| VfsFile::normal(f, filename.to_owned()))
     }