Skip to content

Commit 91b7331

Browse files
std: Directly call unsafe {un,}setenv in env
1 parent 8c75111 commit 91b7331

File tree

1 file changed

+4
-11
lines changed

1 file changed

+4
-11
lines changed

std/src/env.rs

+4-11
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
//! and those without will return a [`String`].
1010
1111
#![stable(feature = "env", since = "1.0.0")]
12-
#![allow(unsafe_op_in_unsafe_fn)]
1312

1413
#[cfg(test)]
1514
mod tests;
@@ -367,11 +366,8 @@ impl Error for VarError {
367366
#[rustc_deprecated_safe_2024]
368367
#[stable(feature = "env", since = "1.0.0")]
369368
pub unsafe fn set_var<K: AsRef<OsStr>, V: AsRef<OsStr>>(key: K, value: V) {
370-
_set_var(key.as_ref(), value.as_ref())
371-
}
372-
373-
unsafe fn _set_var(key: &OsStr, value: &OsStr) {
374-
os_imp::setenv(key, value).unwrap_or_else(|e| {
369+
let (key, value) = (key.as_ref(), value.as_ref());
370+
unsafe { os_imp::setenv(key, value) }.unwrap_or_else(|e| {
375371
panic!("failed to set environment variable `{key:?}` to `{value:?}`: {e}")
376372
})
377373
}
@@ -434,11 +430,8 @@ unsafe fn _set_var(key: &OsStr, value: &OsStr) {
434430
#[rustc_deprecated_safe_2024]
435431
#[stable(feature = "env", since = "1.0.0")]
436432
pub unsafe fn remove_var<K: AsRef<OsStr>>(key: K) {
437-
_remove_var(key.as_ref())
438-
}
439-
440-
unsafe fn _remove_var(key: &OsStr) {
441-
os_imp::unsetenv(key)
433+
let key = key.as_ref();
434+
unsafe { os_imp::unsetenv(key) }
442435
.unwrap_or_else(|e| panic!("failed to remove environment variable `{key:?}`: {e}"))
443436
}
444437

0 commit comments

Comments
 (0)