migrations/egui
1
0
Fork 0
mirror of https://github.com/emilk/egui.git synced 2026-09-01 22:30:03 -04:00
Code Activity

Fix WGPU implementation memory leak

Browse Source 
Now unused surfaces will be cleared
This commit is contained in:
Konkitoman
2023-07-27 18:22:43 +03:00
parent 769c71a714
commit c56d09a2e5
2 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
crates/eframe/src/native/run.rs
View File

@@ -1909,6 +1909,7 @@ mod wgpu_integration {
windows.retain(|id, _| active_viewports_ids.contains(id)); windows.retain(|id, _| active_viewports_ids.contains(id));
windows_id.retain(|_, id| active_viewports_ids.contains(id)); windows_id.retain(|_, id| active_viewports_ids.contains(id));
painter.clean_surfaces(active_viewports_ids);
let mut control_flow = vec![EventResult::Wait]; let mut control_flow = vec![EventResult::Wait];
for repaint_after in repaint_after { for repaint_after in repaint_after {

5
crates/egui-wgpu/src/winit.rs
View File

@@ -615,6 +615,11 @@ impl Painter {
screenshot screenshot
} }
pub fn clean_surfaces(&mut self, avalibile_viewports: Vec<u64>) {
self.surfaces
.retain(|id, _| avalibile_viewports.contains(id))
}
#[allow(clippy::unused_self)] #[allow(clippy::unused_self)]
pub fn destroy(&mut self) { pub fn destroy(&mut self) {
// TODO(emilk): something here? // TODO(emilk): something here?