change map_or_else to unwrap_or_else

This commit is contained in:
Darren Schroeder 2024-06-19 16:12:11 -05:00
parent e1c482ddb9
commit 95c1fbd9b8

View File

@ -194,9 +194,10 @@ pub(crate) fn create_nu_constant(engine_state: &EngineState, span: Span) -> Valu
// if not, use the default /usr/share/nushell/vendor/autoload // if not, use the default /usr/share/nushell/vendor/autoload
// check to see if NU_VENDOR_AUTOLOAD_DIR env var is set, if not, use the default // check to see if NU_VENDOR_AUTOLOAD_DIR env var is set, if not, use the default
Value::string(
option_env!("NU_VENDOR_AUTOLOAD_DIR") option_env!("NU_VENDOR_AUTOLOAD_DIR")
.map(String::from) .map(String::from)
.or_else(|| { .unwrap_or_else(|| {
if cfg!(windows) { if cfg!(windows) {
let all_user_profile = match engine_state.get_env_var("ALLUSERPROFILE") { let all_user_profile = match engine_state.get_env_var("ALLUSERPROFILE") {
Some(v) => format!( Some(v) => format!(
@ -205,18 +206,19 @@ pub(crate) fn create_nu_constant(engine_state: &EngineState, span: Span) -> Valu
), ),
None => "C:\\ProgramData\\nushell\\vendor\\autoload".into(), None => "C:\\ProgramData\\nushell\\vendor\\autoload".into(),
}; };
Some(all_user_profile) all_user_profile
} else { } else {
// In non-Windows environments, if NU_VENDOR_AUTOLOAD_DIR is not set // In non-Windows environments, if NU_VENDOR_AUTOLOAD_DIR is not set
// check to see if PREFIX env var is set, and use it as PREFIX/nushell/vendor/autoload // check to see if PREFIX env var is set, and use it as PREFIX/nushell/vendor/autoload
// otherwise default to /usr/share/nushell/vendor/autoload // otherwise default to /usr/share/nushell/vendor/autoload
option_env!("PREFIX").map(String::from).map_or_else( option_env!("PREFIX").map(String::from).map_or_else(
|| Some("/usr/local/share/nushell/vendor/autoload".into()), || "/usr/local/share/nushell/vendor/autoload".into(),
|prefix| Some(format!("{}/share/nushell/vendor/autoload", prefix)), |prefix| format!("{}/share/nushell/vendor/autoload", prefix),
) )
} }
}) }),
.map_or_else(|| Value::nothing(span), |s| Value::string(s, span)), span,
),
); );
record.push("temp-path", { record.push("temp-path", {