From 5e6cbebcb9616985e5963a55d3e51d6d1e789fa9 Mon Sep 17 00:00:00 2001 From: Richard Stanway Date: Sun, 21 Aug 2022 01:57:36 +0200 Subject: [PATCH] UI: Remove allProjectors list Seems like this wasn't used anywhere and projectors were not being removed on destruction, resulting in a memory leak. --- UI/window-projector.cpp | 5 ----- 1 file changed, 5 deletions(-) diff --git a/UI/window-projector.cpp b/UI/window-projector.cpp index d3f8a34ab..2c1d17359 100644 --- a/UI/window-projector.cpp +++ b/UI/window-projector.cpp @@ -11,7 +11,6 @@ #include "multiview.hpp" static QList multiviewProjectors; -static QList allProjectors; static bool updatingMultiview = false, mouseSwitching, transitionOnDoubleClick; @@ -89,8 +88,6 @@ OBSProjector::OBSProjector(QWidget *widget, obs_source_t *source_, int monitor, if (source) obs_source_inc_showing(source); - allProjectors.push_back(this); - ready = true; show(); @@ -300,8 +297,6 @@ void OBSProjector::EscapeTriggered() { OBSBasic *main = reinterpret_cast(App()->GetMainWindow()); main->DeleteProjector(this); - - allProjectors.removeAll(this); } void OBSProjector::UpdateMultiview()