stupid mistake in import_sync

rust-windowing · Nov 11, 2023 · 3f10cc9 · 3f10cc9
1 parent 814ce28
commit 3f10cc9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/glutin/src/api/egl/display.rs b/glutin/src/api/egl/display.rs
@@ -287,7 +287,7 @@ impl Display {
 
         let import = fd.try_clone_to_owned().map_err(|_| ErrorKind::BadParameter)?;
 
-        let attrs: [egl::types::EGLAttrib; 3] =
+        let attrs: [egl::types::EGLint; 3] =
             [egl::SYNC_NATIVE_FENCE_FD_ANDROID as _, import.as_raw_fd() as _, egl::NONE as _];
 
         // SAFETY:
@@ -298,7 +298,7 @@ impl Display {
             self.inner.egl.CreateSyncKHR(
                 *self.inner.raw,
                 egl::SYNC_NATIVE_FENCE_ANDROID,
-                attrs.as_ptr().cast(),
+                attrs.as_ptr(),
             )
         };
 

diff --git a/glutin_examples/examples/egl_sync.rs b/glutin_examples/examples/egl_sync.rs
@@ -62,7 +62,7 @@ mod example {
         assert!(sync.client_wait(Some(Duration::ZERO), false).unwrap());
 
         // // By default all new Syncs are not signalled.
-        assert!(!sync.is_signalled().unwrap());
+        // assert!(!sync.is_signalled().unwrap());
 
         if display.extensions().contains("EGL_ANDROID_native_fence_sync") {
             println!("EGL Android native fence sync is supported");
@@ -78,7 +78,7 @@ mod example {
             let sync_fd = sync.export_sync_fd().expect("Export failed");
 
             // To show that an exported sync fd can be imported, we will reimport the sync
-            // fd.
+            // fd we just exported.
             let _imported_sync = display.import_sync(sync_fd.as_fd()).expect("Import failed");
         }
     }