feat: bindgen fix and update (#38)

This PR does the following changes:
- update of `bindgen` to `0.72`
- addition of `opaque_type` in bindgen flags to work around alignment
issues with virtual methods in MSFS SDK. This allows `cargo test` to run
successfully.

NOTE: with the update of bindgen from `0.69` to `0.72`, the size and
alignment checks have been moved from unit tests to checks at
compilation time. Without adding `opaque_type`, bindgen 0.69 fails
tests, while bindgen 0.72 fails compilation.

This change was tested with compilation of the a32nx and a380.

Author: Eagle941

msfs/Cargo.toml

@@ -12,6 +12,6 @@ futures = "0.3"
 libc = "0.2"
 
 [build-dependencies]
-bindgen = "0.69"
+bindgen = "0.72"
 msfs_sdk = { path = "../msfs_sdk", version = "0.1.0" }
 cc = "1.0"

msfs/build.rs

@@ -39,7 +39,17 @@ fn main() {
             .blocklist_function("nvgStrokePaint")
             .parse_callbacks(Box::new(bindgen::CargoCallbacks::new()))
             .rustified_enum("SIMCONNECT_EXCEPTION")
-            .impl_debug(false);
+            .impl_debug(false)
+            // `opaque_type` added to avoid alignment errors. These alignment errors are caused
+            // because virtual methods are not well supported in rust-bindgen.
+            .opaque_type("IGaugeCDrawableCreateParameters")
+            .opaque_type("IGaugeCDrawableDrawParameters")
+            .opaque_type("IGaugeCDrawable")
+            .opaque_type("IGaugeCCallback")
+            .opaque_type("ISerializableGaugeCCallback")
+            .opaque_type("IAircraftCCallback")
+            .opaque_type("IPanelCCallback")
+            .opaque_type("IFSXPanelCCallback");
 
         if wasm {
             bindings = bindings.clang_arg("-D_MSFS_WASM 1");