subprocess_launcher: Take owned fds

Author: A6GibKm

gio/src/subprocess_launcher.rs

@@ -1,7 +1,7 @@
 // Take a look at the license at the top of the repository in the LICENSE file.
 
 #[cfg(any(unix, all(docsrs, unix)))]
-use std::os::unix::io::IntoRawFd;
+use std::os::unix::io::{AsFd, AsRawFd, IntoRawFd, OwnedFd};
 
 #[cfg(unix)]
 use glib::translate::*;
@@ -22,40 +22,41 @@ impl SubprocessLauncher {
     #[cfg(unix)]
     #[cfg_attr(docsrs, doc(cfg(unix)))]
     #[doc(alias = "g_subprocess_launcher_take_fd")]
-    pub fn take_fd(&self, source_fd: impl IntoRawFd, target_fd: impl IntoRawFd) {
+    pub fn take_fd(&self, source_fd: OwnedFd, target_fd: impl AsFd) {
+        let source_raw_fd = source_fd.into_raw_fd();
+        let target_raw_fd = target_fd.as_fd().as_raw_fd();
         unsafe {
-            ffi::g_subprocess_launcher_take_fd(
-                self.to_glib_none().0,
-                source_fd.into_raw_fd(),
-                target_fd.into_raw_fd(),
-            );
+            ffi::g_subprocess_launcher_take_fd(self.to_glib_none().0, source_raw_fd, target_raw_fd);
         }
     }
 
     #[cfg(unix)]
     #[cfg_attr(docsrs, doc(cfg(unix)))]
     #[doc(alias = "g_subprocess_launcher_take_stderr_fd")]
-    pub fn take_stderr_fd(&self, fd: impl IntoRawFd) {
+    pub fn take_stderr_fd(&self, fd: Option<OwnedFd>) {
         unsafe {
-            ffi::g_subprocess_launcher_take_stderr_fd(self.to_glib_none().0, fd.into_raw_fd());
+            let raw_fd = fd.map_or(-1, |fd| fd.into_raw_fd());
+            ffi::g_subprocess_launcher_take_stderr_fd(self.to_glib_none().0, raw_fd);
         }
     }
 
     #[cfg(unix)]
     #[cfg_attr(docsrs, doc(cfg(unix)))]
     #[doc(alias = "g_subprocess_launcher_take_stdin_fd")]
-    pub fn take_stdin_fd(&self, fd: impl IntoRawFd) {
+    pub fn take_stdin_fd(&self, fd: Option<OwnedFd>) {
+        let raw_fd = fd.map_or(-1, |fd| fd.into_raw_fd());
         unsafe {
-            ffi::g_subprocess_launcher_take_stdin_fd(self.to_glib_none().0, fd.into_raw_fd());
+            ffi::g_subprocess_launcher_take_stdin_fd(self.to_glib_none().0, raw_fd);
         }
     }
 
     #[cfg(unix)]
     #[cfg_attr(docsrs, doc(cfg(unix)))]
     #[doc(alias = "g_subprocess_launcher_take_stdout_fd")]
-    pub fn take_stdout_fd(&self, fd: impl IntoRawFd) {
+    pub fn take_stdout_fd(&self, fd: Option<OwnedFd>) {
+        let raw_fd = fd.map_or(-1, |fd| fd.into_raw_fd());
         unsafe {
-            ffi::g_subprocess_launcher_take_stdout_fd(self.to_glib_none().0, fd.into_raw_fd());
+            ffi::g_subprocess_launcher_take_stdout_fd(self.to_glib_none().0, raw_fd);
         }
     }
 }