Mark std::process:Child as #[must_use]

Simply dropping this is usually a bad idea, for the reasons extensively discussed in the documentation. Closes: rust-lang#70186 Signed-off-by: Ian Jackson <[email protected]>
ijackson · Jan 28, 2021 · 4921c37 · 4921c37
1 parent be40a3f
commit 4921c37
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/library/std/src/process.rs b/library/std/src/process.rs
@@ -170,6 +170,7 @@ use crate::sys_common::{AsInner, AsInnerMut, FromInner, IntoInner};
 ///
 /// [`wait`]: Child::wait
 #[stable(feature = "process", since = "1.0.0")]
+#[must_use = "this Child should probably be `wait`ed, or `status` used instead of `spawn`"]
 pub struct Child {
     handle: imp::Process,