Move send_ctrlc into inner for doc purposes

Author: nu11ptr

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "send_ctrlc"
-version = "0.1.2"
+version = "0.1.3"
 authors = ["Scott Meeuwsen <smeeuwsen@gmail.com>"]
 license = "MIT OR Apache-2.0"
 description = "A cross platform crate for sending ctrl-c to child processes"

Cargo.toml

@@ -61,10 +61,8 @@ async fn main() {
         child.send_ctrl_c().unwrap();
         child.wait().await.unwrap();
 }
-
 ```
 
-
 ## Contributions
 
 Contributions are welcome as long they align with the vision for this crate.

README.md

@@ -15,38 +15,10 @@ const CREATE_NEW_PROCESS_GROUP: u32 = 0x00000200;
 /// Trait for sending interrupts/ctrl-c to child processes
 pub trait Interruptable: InterruptablePid {
     /// Send a ctrl-c interrupt to the child process
-    #[cfg(all(not(windows), not(unix)))]
-    fn send_ctrl_c(&self) -> io::Result<()> {
-        unimplemented!("Not implemented for this platform");
-    }
-
-    /// Send a ctrl-c interrupt to the child process
-    #[cfg(unix)]
-    fn send_ctrl_c(&self) -> io::Result<()> {
-        if let Some(pid) = self.pid() {
-            if unsafe { libc::kill(pid as i32, libc::SIGINT) } == 0 {
-                Ok(())
-            } else {
-                Err(io::Error::last_os_error())
-            }
-        } else {
-            Err(io::Error::new(io::ErrorKind::Other, "Process has no pid"))
-        }
-    }
-
-    /// Send a ctrl-c interrupt to the child process
-    #[cfg(windows)]
     fn send_ctrl_c(&self) -> io::Result<()> {
-        use windows_sys::Win32::System::Console::{CTRL_C_EVENT, GenerateConsoleCtrlEvent};
-        if let Some(pid) = self.pid() {
-            // NOTE: This only works if the process is in a new process group
-            if unsafe { GenerateConsoleCtrlEvent(CTRL_C_EVENT, pid) } != 0 {
-                Ok(())
-            } else {
-                Err(io::Error::last_os_error())
-            }
-        } else {
-            Err(io::Error::new(io::ErrorKind::Other, "Process has no pid"))
+        match self.pid() {
+            Some(pid) => inner::send_ctrl_c(pid),
+            None => Err(io::Error::new(io::ErrorKind::Other, "Process has no pid")),
         }
     }
 }
@@ -78,7 +50,33 @@ pub fn new_command<S: AsRef<OsStr>>(program: S) -> Command {
 }
 
 mod inner {
-    use std::{ffi::OsStr, process::Command};
+    use std::{ffi::OsStr, io, process::Command};
+
+    #[cfg(all(not(windows), not(unix)))]
+    pub fn send_ctrl_c(_pid: u32) -> io::Result<()> {
+        unimplemented!("Not implemented for this platform");
+    }
+
+    #[cfg(unix)]
+    pub fn send_ctrl_c(pid: u32) -> io::Result<()> {
+        if unsafe { libc::kill(pid as i32, libc::SIGINT) } == 0 {
+            Ok(())
+        } else {
+            Err(io::Error::last_os_error())
+        }
+    }
+
+    #[cfg(windows)]
+    pub fn send_ctrl_c(pid: u32) -> io::Result<()> {
+        use windows_sys::Win32::System::Console::{CTRL_C_EVENT, GenerateConsoleCtrlEvent};
+
+        // NOTE: This only works if the process is in a new process group
+        if unsafe { GenerateConsoleCtrlEvent(CTRL_C_EVENT, pid) } != 0 {
+            Ok(())
+        } else {
+            Err(io::Error::last_os_error())
+        }
+    }
 
     #[cfg(windows)]
     pub fn new_command<S: AsRef<OsStr>>(program: S) -> Command {
@@ -98,7 +96,6 @@ mod inner {
 }
 
 /// Create a new interruptable tokio command
-#[cfg_attr(docsrs, doc(cfg(feature = "tokio")))]
 #[cfg(feature = "tokio")]
 pub fn new_tokio_command<S: AsRef<OsStr>>(program: S) -> tokio::process::Command {
     inner_tokio::new_tokio_command(program)