settings: implement strv setter and getter manually

Author: pbor

gio/Gir.toml

@@ -1114,6 +1114,14 @@ status = "generate"
         [object.function.return]
         bool_return_is_error = "Can't set readonly key"
     [[object.function]]
+    name = "get_strv"
+    # Use strv
+    manual = true
+    [[object.function]]
+    name = "set_strv"
+    # Use strv
+    manual = true
+    [[object.function]]
     name = "bind"
     manual = true
     [[object.function]]

gio/src/auto/settings.rs

@@ -162,10 +162,6 @@ pub trait SettingsExt: 'static {
     #[doc(alias = "get_string")]
     fn string(&self, key: &str) -> glib::GString;
 
-    #[doc(alias = "g_settings_get_strv")]
-    #[doc(alias = "get_strv")]
-    fn strv(&self, key: &str) -> Vec<glib::GString>;
-
     #[doc(alias = "g_settings_get_uint")]
     #[doc(alias = "get_uint")]
     fn uint(&self, key: &str) -> u32;
@@ -218,9 +214,6 @@ pub trait SettingsExt: 'static {
     #[doc(alias = "g_settings_set_string")]
     fn set_string(&self, key: &str, value: &str) -> Result<(), glib::error::BoolError>;
 
-    #[doc(alias = "g_settings_set_strv")]
-    fn set_strv(&self, key: &str, value: &[&str]) -> Result<(), glib::error::BoolError>;
-
     #[doc(alias = "g_settings_set_uint")]
     fn set_uint(&self, key: &str, value: u32) -> Result<(), glib::error::BoolError>;
 
@@ -385,15 +378,6 @@ impl<O: IsA<Settings>> SettingsExt for O {
         }
     }
 
-    fn strv(&self, key: &str) -> Vec<glib::GString> {
-        unsafe {
-            FromGlibPtrContainer::from_glib_full(ffi::g_settings_get_strv(
-                self.as_ref().to_glib_none().0,
-                key.to_glib_none().0,
-            ))
-        }
-    }
-
     fn uint(&self, key: &str) -> u32 {
         unsafe { ffi::g_settings_get_uint(self.as_ref().to_glib_none().0, key.to_glib_none().0) }
     }
@@ -544,19 +528,6 @@ impl<O: IsA<Settings>> SettingsExt for O {
         }
     }
 
-    fn set_strv(&self, key: &str, value: &[&str]) -> Result<(), glib::error::BoolError> {
-        unsafe {
-            glib::result_from_gboolean!(
-                ffi::g_settings_set_strv(
-                    self.as_ref().to_glib_none().0,
-                    key.to_glib_none().0,
-                    value.to_glib_none().0
-                ),
-                "Can't set readonly key"
-            )
-        }
-    }
-
     fn set_uint(&self, key: &str, value: u32) -> Result<(), glib::error::BoolError> {
         unsafe {
             glib::result_from_gboolean!(

gio/src/settings.rs

@@ -1,6 +1,6 @@
 // Take a look at the license at the top of the repository in the LICENSE file.
 
-use glib::{prelude::*, translate::*, variant::FromVariant, BoolError, Variant};
+use glib::{prelude::*, translate::*, variant::FromVariant, BoolError, IntoStrV, StrV, Variant};
 
 use crate::{prelude::*, Settings, SettingsBindFlags};
 
@@ -170,6 +170,13 @@ pub trait SettingsExtManual {
 
     fn set(&self, key: &str, value: impl Into<Variant>) -> Result<(), BoolError>;
 
+    #[doc(alias = "g_settings_get_strv")]
+    #[doc(alias = "get_strv")]
+    fn strv(&self, key: &str) -> StrV;
+
+    #[doc(alias = "g_settings_set_strv")]
+    fn set_strv(&self, key: &str, value: impl IntoStrV) -> Result<(), glib::error::BoolError>;
+
     #[doc(alias = "g_settings_bind")]
     #[doc(alias = "g_settings_bind_with_mapping")]
     fn bind<'a, P: IsA<glib::Object>>(
@@ -196,6 +203,30 @@ impl<O: IsA<Settings>> SettingsExtManual for O {
         self.set_value(key, &value.into())
     }
 
+    fn strv(&self, key: &str) -> StrV {
+        unsafe {
+            FromGlibPtrContainer::from_glib_full(ffi::g_settings_get_strv(
+                self.as_ref().to_glib_none().0,
+                key.to_glib_none().0,
+            ))
+        }
+    }
+
+    fn set_strv(&self, key: &str, value: impl IntoStrV) -> Result<(), glib::error::BoolError> {
+        unsafe {
+            value.run_with_strv(|value| {
+                glib::result_from_gboolean!(
+                    ffi::g_settings_set_strv(
+                        self.as_ref().to_glib_none().0,
+                        key.to_glib_none().0,
+                        value.as_ptr() as *mut _,
+                    ),
+                    "Can't set readonly key"
+                )
+            })
+        }
+    }
+
     fn bind<'a, P: IsA<glib::Object>>(
         &'a self,
         key: &'a str,