Make bind_image_texture take Option<Self::Texture> (#325)

Fix #259 Signed-off-by: sagudev <16504129+sagudev@users.noreply.github.com>
grovesNL · Nov 5, 2024 · 36dc9f4 · 36dc9f4
1 parent 62cfeee
commit 36dc9f4
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/lib.rs b/src/lib.rs
@@ -1900,7 +1900,7 @@ pub trait HasContext: __private::Sealed {
     unsafe fn bind_image_texture(
         &self,
         unit: u32,
-        texture: Self::Texture,
+        texture: Option<Self::Texture>,
         level: i32,
         layered: bool,
         layer: i32,

diff --git a/src/native.rs b/src/native.rs
@@ -4342,7 +4342,7 @@ impl HasContext for Context {
     unsafe fn bind_image_texture(
         &self,
         unit: u32,
-        texture: Self::Texture,
+        texture: Option<Self::Texture>,
         level: i32,
         layered: bool,
         layer: i32,
@@ -4352,7 +4352,7 @@ impl HasContext for Context {
         let gl = &self.raw;
         gl.BindImageTexture(
             unit,
-            texture.0.get(),
+            texture.map(|tex| tex.0.get()).unwrap_or(0),
             level,
             layered as u8,
             layer,

diff --git a/src/web_sys.rs b/src/web_sys.rs
@@ -6035,7 +6035,7 @@ impl HasContext for Context {
     unsafe fn bind_image_texture(
         &self,
         _unit: u32,
-        _texture: Self::Texture,
+        _texture: Option<Self::Texture>,
         _level: i32,
         _layered: bool,
         _layer: i32,