From 6ce48419b6e09d32c39410fffb3d532251ff5867 Mon Sep 17 00:00:00 2001 From: Jiahao XU Date: Fri, 22 Jul 2022 22:55:44 +1000 Subject: [PATCH] Fix `CratesToml::write_to_{writer, file}` Make them consistent with `Crates2Json::write_to_{writer, file}` Signed-off-by: Jiahao XU --- src/metafiles/v1.rs | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/src/metafiles/v1.rs b/src/metafiles/v1.rs index 79126091..def69db6 100644 --- a/src/metafiles/v1.rs +++ b/src/metafiles/v1.rs @@ -47,18 +47,16 @@ impl CratesToml { self.write_to_path(Self::default_path()?) } - pub fn write_to_writer( - &self, - mut writer: W, - ) -> Result { + pub fn write_to_writer(&self, mut writer: W) -> Result<(), CratesTomlParseError> { let data = toml::to_vec(&self)?; writer.write_all(&data)?; - Ok(data.len().try_into().unwrap()) + Ok(()) } pub fn write_to_file(&self, file: &mut fs::File) -> Result<(), CratesTomlParseError> { - let cnt = self.write_to_writer(&mut *file)?; - file.set_len(cnt)?; + self.write_to_writer(&mut *file)?; + let pos = file.stream_position()?; + file.set_len(pos)?; Ok(()) }