bind_image_texture should take `Option<Self::Texture>`

grovesNL / glow

GL on Whatever: a set of bindings to run GL anywhere and avoid target-specific code

Apache License 2.0

1.17k stars 130 forks source link

bind_image_texture should take `Option<Self::Texture>` #259

Open ennis opened 11 months ago

ennis commented 11 months ago

It's valid to call glBindImageTexture with texture equal to zero, in which case it will break any previous binding. The current signature takes Self::Texture. It should be Option<Self::Texture> to be consistent with other bind_ functions.

grovesNL commented 11 months ago

Sure, happy to accept a PR for this