Remove wait_attach_auto_detect, trust asr version

latest patch uses V3 now, so the assumption behind the old wait_attach_auto_detect no longer holds

Author: AlexKnauth

src/hollow_knight_memory.rs

@@ -2519,7 +2519,7 @@ impl GameManagerFinder {
         let mut found_module = false;
         let mut needed_retry = false;
         loop {
-            let module = wait_attach_auto_detect(process).await;
+            let module = mono::Module::wait_attach_auto_detect(process).await;
             if !found_module {
                 found_module = true;
                 asr::print_message("GameManagerFinder wait_attach: module get_default_image...");
@@ -7409,24 +7409,6 @@ pub fn attach_hollow_knight() -> Option<Process> {
     HOLLOW_KNIGHT_NAMES.into_iter().find_map(Process::attach)
 }
 
-async fn wait_attach_auto_detect(process: &Process) -> mono::Module {
-    retry(|| attach_auto_detect(process)).await
-}
-
-fn attach_auto_detect(process: &Process) -> Option<mono::Module> {
-    if process.get_module_address("mono.dll").is_ok()
-        || process.get_module_address("libmono.so").is_ok()
-        || process.get_module_address("libmono.0.dylib").is_ok()
-    {
-        // V1 or V1Cattrs
-        mono::Module::attach_auto_detect(process)
-    } else {
-        // if it's not V1 or V1Cattrs,
-        // it's always V2, never V3
-        mono::Module::attach(process, mono::Version::V2)
-    }
-}
-
 fn scene_path_to_name_string<const N: usize>(scene_path: ArrayCString<N>) -> Option<String> {
     String::from_utf8(scene_manager::get_name(&scene_path).to_vec()).ok()
 }