ndk: Add ASurfaceControl bindings

Author: MarijnS95

ndk/CHANGELOG.md

@@ -1,6 +1,7 @@
 # Unreleased
 
 - image_reader: Add `ImageReader::new_with_data_space()` constructor and `ImageReader::data_space()` getter from API level 34. (#474)
+- Add `ASurfaceControl` and `ASurfaceTransaction` bindings. (#TODO)
 
 # 0.9.0 (2024-04-26)
 

ndk/src/data_space.rs

@@ -236,6 +236,18 @@ pub enum DataSpace {
     __Unknown(i32),
 }
 
+impl From<DataSpace> for ffi::ADataSpace {
+    fn from(val: DataSpace) -> Self {
+        Self(val.into())
+    }
+}
+
+impl From<ffi::ADataSpace> for DataSpace {
+    fn from(val: ffi::ADataSpace) -> Self {
+        Self::from(val.0)
+    }
+}
+
 impl fmt::Display for DataSpace {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         f.write_str(match self {

ndk/src/lib.rs

@@ -27,6 +27,7 @@ pub mod media_error;
 pub mod native_activity;
 pub mod native_window;
 pub mod shared_memory;
+pub mod surface_control;
 pub mod surface_texture;
 pub mod sync;
 pub mod trace;

ndk/src/native_window.rs

@@ -204,7 +204,7 @@ impl NativeWindow {
     /// may choose a display refresh rate to better match this window's frame rate. Usage of this
     /// API won't introduce frame rate throttling, or affect other aspects of the application's
     /// frame production pipeline. However, because the system may change the display refresh rate,
-    /// calls to this function may result in changes to Choreographer callback timings, and changes
+    /// calls to this function may result in changes to `Choreographer` callback timings, and changes
     /// to the time interval at which the system releases buffers back to the application.
     ///
     /// Note that this only has an effect for windows presented on the display. If this
@@ -390,7 +390,7 @@ impl Drop for NativeWindowBufferLockGuard<'_> {
     }
 }
 
-#[cfg(all(feature = "nativewindow", feature = "api-level-26"))]
+#[cfg(feature = "api-level-26")]
 bitflags::bitflags! {
     /// Transforms that can be applied to buffers as they are displayed to a window.
     ///
@@ -426,7 +426,7 @@ bitflags::bitflags! {
     doc = " and [`NativeWindow::set_frame_rate_with_change_strategy()`]"
 )]
 /// .
-#[cfg(all(feature = "nativewindow", feature = "api-level-30"))]
+#[cfg(feature = "api-level-30")]
 #[repr(i8)]
 #[derive(Clone, Copy, Debug, Hash, PartialEq, Eq)]
 #[doc(alias = "ANativeWindow_FrameRateCompatibility")]
@@ -453,7 +453,7 @@ pub enum FrameRateCompatibility {
 }
 
 /// Change frame rate strategy value for [`NativeWindow::set_frame_rate_with_change_strategy()`].
-#[cfg(all(feature = "nativewindow", feature = "api-level-31"))]
+#[cfg(feature = "api-level-31")]
 #[repr(i8)]
 #[derive(Clone, Copy, Debug, Hash, PartialEq, Eq)]
 #[doc(alias = "ANativeWindow_ChangeFrameRateStrategy")]