Fixed warnings addded

Author: IberAI

src/install/manifest.rs

@@ -37,8 +37,7 @@ struct FirefoxHostManifest<'a> {
 }
 
 fn ensure_absolute_path(exe_path: &Path) -> io::Result<()> {
-    // Chrome spec: path must be absolute on macOS/Linux. Firefox follows same convention.
-    // Windows allows relative, but absolute is safest cross-platform. :contentReference[oaicite:3]{index=3}
+    // On macOS/Linux the manifest "path" MUST be absolute.
     #[cfg(any(target_os = "macos", target_os = "linux"))]
     {
         if !exe_path.is_absolute() {
@@ -48,6 +47,14 @@ fn ensure_absolute_path(exe_path: &Path) -> io::Result<()> {
             ));
         }
     }
+
+    // On Windows we accept any (absolute recommended). Touch exe_path
+    // to avoid an unused-variable warning in Windows builds.
+    #[cfg(windows)]
+    {
+        let _ = exe_path;
+    }
+
     Ok(())
 }
 

src/install/winreg.rs

@@ -1,10 +1,9 @@
-#![cfg(windows)]
-#![cfg_attr(not(windows), allow(dead_code))]
-
 use crate::install::paths::chrome_user_manifest;
 use std::{io, path::PathBuf};
 use winreg::{enums::HKEY_CURRENT_USER, RegKey};
 
+/// Write the Chrome native-messaging registry value under HKCU so Chrome
+/// can find the manifest file at the given host `name`.
 pub fn write_chrome_manifest_reg(name: &str) -> io::Result<()> {
     let hkcu = RegKey::predef(HKEY_CURRENT_USER);
     let key_path = format!(r"Software\Google\Chrome\NativeMessagingHosts\{name}");
@@ -14,6 +13,7 @@ pub fn write_chrome_manifest_reg(name: &str) -> io::Result<()> {
     Ok(())
 }
 
+/// Remove the HKCU registry value for the Chrome native-messaging host.
 pub fn remove_chrome_manifest_reg(name: &str) -> io::Result<()> {
     let hkcu = RegKey::predef(HKEY_CURRENT_USER);
     let key_path = format!(r"Software\Google\Chrome\NativeMessagingHosts\{name}");