Re: [PATCH 0/1] MR7717: Initial patch for converting Wine to Rust

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about tools/wine/wine.rs:
> -
>            "unix"
> -
>        }
> -
>    )
> - } else {
> -
>    "".to_string()
> - }
>
> +}
> -
>
> +fn is_pe_target(_target: &Target) -> bool {
> - // Implement platform-specific PE detection logic if needed
> - false
>
> +}
> -
>
> +fn load_ntdll(bindir_in: Option<String>, libdir_in: Option<String>) -> Library {
> - let arch_dir = get_arch_dir(&get_default_target());
> - let winedllpath = env::var("WINEDLLPATH").unwrap_or_default();

Is using Option an option?