explicit drop semantics

Author: extrawurst

git2-hooks/src/hookspath.rs

@@ -176,11 +176,12 @@ impl HookPaths {
 				.stderr(std::process::Stdio::piped())
 				.spawn()?;
 
-			if let Some(input) = stdin {
+			if let Some(mut stdin_handle) = child.stdin.take() {
 				use std::io::Write;
-				if let Some(mut stdin_handle) = child.stdin.take() {
+				if let Some(input) = stdin {
 					stdin_handle.write_all(input)?;
 				}
+				drop(stdin_handle);
 			}
 
 			child.wait_with_output()

src/clipboard.rs

@@ -26,12 +26,15 @@ fn exec_copy_with_args(
 		.spawn()
 		.map_err(|e| anyhow!("`{command:?}`: {e:?}"))?;
 
-	process
+	let mut stdin = process
 		.stdin
-		.as_mut()
-		.ok_or_else(|| anyhow!("`{command:?}`"))?
+		.take()
+		.ok_or_else(|| anyhow!("`{command:?}`"))?;
+
+	stdin
 		.write_all(text.as_bytes())
 		.map_err(|e| anyhow!("`{command:?}`: {e:?}"))?;
+	drop(stdin);
 
 	let out = process
 		.wait_with_output()