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 crash on window minimize on vulkan renderer. #2204

Merged
merged 1 commit into from
Jul 18, 2020

Conversation

kingscallop
Copy link
Contributor

On some platforms (Windows) the framebuffer size becomes 0x0 when
the window is minimized. This leads to a validation error and
subsequently crashes.

This change correctly cleans up semaphores and the swapchain.

On some platforms (Windows) the framebuffer size becomes 0x0 when
the window is minimized. This leads to a validation error and
subsequently crashes.

This change correctly cleans up semaphores and the swapchain.
@bkaradzic bkaradzic merged commit 6093250 into bkaradzic:master Jul 18, 2020
rozgo pushed a commit to VertexStudio/bgfx that referenced this pull request Aug 14, 2020
On some platforms (Windows) the framebuffer size becomes 0x0 when
the window is minimized. This leads to a validation error and
subsequently crashes.

This change correctly cleans up semaphores and the swapchain.
pigpigyyy added a commit to pigpigyyy/bgfx that referenced this pull request Mar 11, 2021
* master: (21 commits)
  Improves handling of window resizes on vulkan (avoids crashes) (bkaradzic#2123)
  Fix crash on window minimize on vulkan renderer (bkaradzic#2204)
  Fix X11 flicker when window is being resized (bkaradzic#2203)
  Updated spirv-cross.
  Updated spirv-tools.
  Updated glslang.
  Updated ImGui.
  Cleanup.
  Fixed docs.
  Fixed issue bkaradzic#2201.
  Updated version number.
  Cleanup.
  OpenGL: Disable scissor testing while blitting framebuffers when MSAA is enabled (bkaradzic#2200)
  Updating xcode config. (bkaradzic#2198)
  Updated glslang.
  shaderc: Disabled warnings.
  Updated spirv-cross.
  Updated spirv-tools.
  Updated spirv-headers.
  Updated vulkan headers.
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants