Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix WGPUInstanceEnumerateAdapterOptions, wgpuSwapChainGetCurrentTextureView & wgpuDevicePopErrorScope #276

Merged
merged 4 commits into from
Jul 6, 2023

Conversation

rajveermalviya
Copy link
Collaborator

@rajveermalviya rajveermalviya commented Jul 4, 2023

Changes

  • WGPUInstanceEnumerateAdapterOptions should take nextInChain pointer instead of chain.
  • wgpuDevicePopErrorScope should always call the callback, even when no error occured in the error scope.
  • correctly handle wgpuSwapChainGetCurrentTextureView errors in the triangle example.
  • fix a potential leak of texture object in wgpuSwapChainGetCurrentTextureView.

Testing

ran the examples on linux

fixes #275
fixes #267

The callback must be called with `WGPUErrorType_NoError` type,
when no error has occured.
fixes a potential leak of texture object.
@rajveermalviya rajveermalviya merged commit 523c6b3 into gfx-rs:trunk Jul 6, 2023
15 checks passed
@rajveermalviya rajveermalviya deleted the bugs branch July 6, 2023 03:14
Beyley pushed a commit to Beyley/wgpu-native that referenced this pull request Jul 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants