[pdm-devel] [PATCH proxmox v2 1/3] section-config: make write_section_config parameter more generic

Gabriel Goller g.goller at proxmox.com
Wed Apr 23 12:20:42 CEST 2025


The underlying function which gets called by `write_section_config` also
takes a `impl AsRef<Path>` and in some cases we do pass a path, so
taking the same parameter is better.

Signed-off-by: Gabriel Goller <g.goller at proxmox.com>
---
 proxmox-section-config/src/typed.rs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/proxmox-section-config/src/typed.rs b/proxmox-section-config/src/typed.rs
index 31e8e4b2e546..271bad3ad057 100644
--- a/proxmox-section-config/src/typed.rs
+++ b/proxmox-section-config/src/typed.rs
@@ -1,6 +1,7 @@
 //! Support for `enum` typed section configs.
 
 use std::collections::HashMap;
+use std::path::Path;
 
 use anyhow::Error;
 use serde::{de::DeserializeOwned, Serialize};
@@ -90,9 +91,10 @@ pub trait ApiSectionDataEntry: Sized {
     }
 
     /// Provided. Shortcut for `Self::section_config().write(filename, &data.try_into()?)`.
-    fn write_section_config(filename: &str, data: &SectionConfigData<Self>) -> Result<String, Error>
+    fn write_section_config<P>(filename: P, data: &SectionConfigData<Self>) -> Result<String, Error>
     where
         Self: Serialize,
+        P: AsRef<Path>,
     {
         Self::section_config().write(filename, &data.try_into()?)
     }
-- 
2.39.5





More information about the pdm-devel mailing list