diff options
author | 2024-10-28 07:50:17 +0000 | |
---|---|---|
committer | 2024-10-28 07:50:17 +0000 | |
commit | bcd6336d7f03fb6884a537032a1a96398e677b0b (patch) | |
tree | b8804d04651e26864f9e5854890743cc40f3960b | |
parent | d3e05e7fd18329c3e0be4297d6d68de8f98a38dd (diff) | |
parent | 7e947d331d83cb732b8039f106674f072202cf16 (diff) |
Merge "Improve formatting of doc comments and add more details." into main
-rw-r--r-- | libs/nativewindow/rust/src/lib.rs | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/libs/nativewindow/rust/src/lib.rs b/libs/nativewindow/rust/src/lib.rs index a1d986ed2b..014c912a67 100644 --- a/libs/nativewindow/rust/src/lib.rs +++ b/libs/nativewindow/rust/src/lib.rs @@ -203,8 +203,8 @@ impl HardwareBuffer { Self(buffer_ptr) } - /// Creates a new Rust HardwareBuffer to wrap the given AHardwareBuffer without taking ownership - /// of it. + /// Creates a new Rust HardwareBuffer to wrap the given `AHardwareBuffer` without taking + /// ownership of it. /// /// Unlike [`from_raw`](Self::from_raw) this method will increment the refcount on the buffer. /// This means that the caller can continue to use the raw buffer it passed in, and must call @@ -220,14 +220,20 @@ impl HardwareBuffer { Self(buffer) } - /// Get the internal `AHardwareBuffer` pointer that is only valid when this `HardwareBuffer` - /// exists. This can be used to provide a pointer to the AHB for a C/C++ API over the FFI. + /// Returns the internal `AHardwareBuffer` pointer. + /// + /// This is only valid as long as this `HardwareBuffer` exists, so shouldn't be stored. It can + /// be used to provide a pointer for a C/C++ API over FFI. pub fn as_raw(&self) -> NonNull<AHardwareBuffer> { self.0 } - /// Get the internal `AHardwareBuffer` pointer without decrementing the refcount. This can - /// be used to provide a pointer to the AHB for a C/C++ API over the FFI. + /// Gets the internal `AHardwareBuffer` pointer without decrementing the refcount. This can + /// be used for a C/C++ API which takes ownership of the pointer. + /// + /// The caller is responsible for releasing the `AHardwareBuffer` pointer by calling + /// `AHardwareBuffer_release` when it is finished with it, or may convert it back to a Rust + /// `HardwareBuffer` by calling [`HardwareBuffer::from_raw`]. pub fn into_raw(self) -> NonNull<AHardwareBuffer> { let buffer = ManuallyDrop::new(self); buffer.0 |