fix(rust): Fix Rust bindings compilation on Windows

- Fix formatting issues (trailing spaces in doc comments)
- Fix Windows MSVC linker errors by:
  * Always compiling ccap_convert_neon.cpp (provides hasNEON() symbol)
  * Using MSVC-compatible flags (/arch:AVX2) for AVX2 builds
  * Removing -mavx2/-mfma flags that are GCC/Clang specific

The hasNEON() function is needed on all platforms (returns false on non-ARM).
Previously it was only compiled on aarch64, causing undefined symbol errors
on Windows x86_64.

Author: wysaid

bindings/rust/build.rs

@@ -90,22 +90,37 @@ fn main() {
                 .include(ccap_root.join("include"))
                 .include(ccap_root.join("src"))
                 .cpp(true)
-                .std("c++17")
-                .flag("-mavx2")
-                .flag("-mfma")
-                .compile("ccap_avx2");
+                .std("c++17");
+
+            // Only add SIMD flags on non-MSVC compilers
+            if !avx2_build.get_compiler().is_like_msvc() {
+                avx2_build.flag("-mavx2").flag("-mfma");
+            } else {
+                // MSVC uses /arch:AVX2
+                avx2_build.flag("/arch:AVX2");
+            }
+
+            avx2_build.compile("ccap_avx2");
         }
 
-        #[cfg(target_arch = "aarch64")]
+        // Always build neon file for hasNEON() symbol
+        // On non-ARM architectures, ENABLE_NEON_IMP will be 0 and function returns false
         {
             let mut neon_build = cc::Build::new();
             neon_build
                 .file(ccap_root.join("src/ccap_convert_neon.cpp"))
                 .include(ccap_root.join("include"))
                 .include(ccap_root.join("src"))
                 .cpp(true)
-                .std("c++17")
-                .compile("ccap_neon");
+                .std("c++17");
+
+            // Only add NEON flags on aarch64
+            #[cfg(target_arch = "aarch64")]
+            {
+                // NEON is always available on aarch64, no special flags needed
+            }
+
+            neon_build.compile("ccap_neon");
         }
 
         println!("cargo:warning=Building ccap from source...");

bindings/rust/src/provider.rs

@@ -328,13 +328,13 @@ impl Provider {
     }
 
     /// Set error callback for camera errors
-    /// 
+    ///
     /// # Memory Safety
-    /// 
+    ///
     /// This is a **global** callback that persists for the lifetime of the program.
     /// The callback memory is intentionally leaked as it's meant to be set once
     /// and used throughout the application lifetime.
-    /// 
+    ///
     /// If you need to change or remove the callback, consider using instance-level
     /// callbacks via `set_new_frame_callback` instead.
     pub fn set_error_callback<F>(callback: F)
@@ -453,7 +453,9 @@ impl Provider {
             unsafe {
                 let _ = Box::from_raw(callback_ptr);
             }
-            Err(CcapError::InvalidParameter("Failed to set frame callback".to_string()))
+            Err(CcapError::InvalidParameter(
+                "Failed to set frame callback".to_string(),
+            ))
         }
     }