From 4c9ebc753ca6db7bd3b86e5d2d51c182a50d8530 Mon Sep 17 00:00:00 2001 From: gxalpha Date: Fri, 10 Mar 2023 14:47:43 +0100 Subject: [PATCH] UI: Create OBSPermissions on stack This fixes the window leaking and may improve performance. --- UI/obs-app.cpp | 8 ++++---- UI/window-basic-main.cpp | 11 +++++------ 2 files changed, 9 insertions(+), 10 deletions(-) diff --git a/UI/obs-app.cpp b/UI/obs-app.cpp index cf293caf7..860ce400a 100644 --- a/UI/obs-app.cpp +++ b/UI/obs-app.cpp @@ -2442,10 +2442,10 @@ static int run_program(fstream &logFile, int argc, char *argv[]) "MacOSPermissionsDialogLastShown"); if (permissionsDialogLastShown < MACOS_PERMISSIONS_DIALOG_VERSION) { - OBSPermissions *check = new OBSPermissions( - nullptr, screen_permission, video_permission, - audio_permission, accessibility_permission); - check->exec(); + OBSPermissions check(nullptr, screen_permission, + video_permission, audio_permission, + accessibility_permission); + check.exec(); } #endif diff --git a/UI/window-basic-main.cpp b/UI/window-basic-main.cpp index dbd3431cc..4a0860a8d 100644 --- a/UI/window-basic-main.cpp +++ b/UI/window-basic-main.cpp @@ -5033,12 +5033,11 @@ void OBSBasic::on_action_Settings_triggered() void OBSBasic::on_actionShowMacPermissions_triggered() { #ifdef __APPLE__ - OBSPermissions *check = - new OBSPermissions(this, CheckPermission(kScreenCapture), - CheckPermission(kVideoDeviceAccess), - CheckPermission(kAudioDeviceAccess), - CheckPermission(kAccessibility)); - check->exec(); + OBSPermissions check(this, CheckPermission(kScreenCapture), + CheckPermission(kVideoDeviceAccess), + CheckPermission(kAudioDeviceAccess), + CheckPermission(kAccessibility)); + check.exec(); #endif }