claude_code/procedural-3d-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #907 from rajnesh-kanwal/fix_memory_leak

Browse source 
Fixes memory leak due to ImGui context not being destroyed.
This commit is contained in:
Sascha Willems 2021-12-11 11:00:13 +01:00 • committed by GitHub
commit 48de215d0b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
base/VulkanUIOverlay.cpp
View file

@ -50,7 +50,9 @@ namespace vks
io.FontGlobalScale = scale; io.FontGlobalScale = scale;
} }
UIOverlay::~UIOverlay() { } UIOverlay::~UIOverlay() {
ImGui::DestroyContext();
}
/** Prepare all vulkan resources required to render the UI overlay */ /** Prepare all vulkan resources required to render the UI overlay */
void UIOverlay::prepareResources() void UIOverlay::prepareResources()
@ -397,7 +399,6 @@ namespace vks
void UIOverlay::freeResources() void UIOverlay::freeResources()
{ {
ImGui::DestroyContext();
vertexBuffer.destroy(); vertexBuffer.destroy();
indexBuffer.destroy(); indexBuffer.destroy();
vkDestroyImageView(device->logicalDevice, fontView, nullptr); vkDestroyImageView(device->logicalDevice, fontView, nullptr);