impl IoSafe for std::io::PipeReader & std::io::PipeWriter

It would be handy to have async wrappers for the pipe types (from
std::io::pipe()), so impl IoSafe for both.

The pipe types were introduced in 1.87, but the MSRV for async-io
appears to be 1.63 currently. So, protect the impls with cfg()s
and use autocfg to detect the type availbility, similar to what was done
in a20076f ("Implement I/O-safe traits (#84)")

Signed-off-by: Jeremy Kerr <[email protected]>

Author: jk-ozlabs

Cargo.toml

@@ -40,6 +40,9 @@ tracing = { version = "0.1.37", default-features = false, optional = true }
 [target.'cfg(windows)'.dependencies]
 windows-sys = { version = "0.60.0", features = ["Win32_Foundation"] }
 
+[build-dependencies]
+autocfg = "1"
+
 [dev-dependencies]
 async-channel = "2.0.0"
 async-net = "2.0.0"

build.rs

@@ -0,0 +1,17 @@
+fn main() {
+    let cfg = match autocfg::AutoCfg::new() {
+        Ok(cfg) => cfg,
+        Err(e) => {
+            println!("cargo:warning=async-io: failed to detect compiler features: {e}");
+            return;
+        }
+    };
+
+    // emit expected config settings; may be replaced with autocfg::emit_possibility
+    // if we update to autocfg 1.5
+    println!("cargo:rustc-check-cfg=cfg(has_std__io__PipeWriter)");
+    println!("cargo:rustc-check-cfg=cfg(has_std__io__PipeReader)");
+
+    cfg.emit_has_type("std::io::PipeReader");
+    cfg.emit_has_type("std::io::PipeWriter");
+}

src/lib.rs

@@ -1252,6 +1252,13 @@ unsafe impl IoSafe for std::process::ChildStderr {}
 #[cfg(unix)]
 unsafe impl IoSafe for std::os::unix::net::UnixStream {}
 
+// PipeReader & PipeWriter require std >= 1.87, our MSRV is 1.63, hence
+// conditional on cfg()s, generated from build.rs
+#[cfg(has_std__io__PipeReader)]
+unsafe impl IoSafe for std::io::PipeReader {}
+#[cfg(has_std__io__PipeWriter)]
+unsafe impl IoSafe for std::io::PipeWriter {}
+
 unsafe impl<T: IoSafe + Read> IoSafe for std::io::BufReader<T> {}
 unsafe impl<T: IoSafe + Write> IoSafe for std::io::BufWriter<T> {}
 unsafe impl<T: IoSafe + Write> IoSafe for std::io::LineWriter<T> {}