From e511718fbc9207284bc5d109bc1d958476fe104b Mon Sep 17 00:00:00 2001 From: "Michael Cook (mackal)" <277429+mackal@users.noreply.github.com> Date: Tue, 4 Apr 2023 16:05:23 -0400 Subject: [PATCH] DolphinQt: Fix leak in LogConfigWidget The m_verbosity_debug button was only conditionally being added as widget, this was done in order to hide the object, but this left it unmanaged. Unconditionally adding it to the layout and controlling it's visibility will resolve these issues --- Source/Core/DolphinQt/Config/LogConfigWidget.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/Source/Core/DolphinQt/Config/LogConfigWidget.cpp b/Source/Core/DolphinQt/Config/LogConfigWidget.cpp index dab59e63d9..607d3618c2 100644 --- a/Source/Core/DolphinQt/Config/LogConfigWidget.cpp +++ b/Source/Core/DolphinQt/Config/LogConfigWidget.cpp @@ -47,6 +47,7 @@ void LogConfigWidget::CreateWidgets() m_verbosity_warning = new QRadioButton(tr("Warning")); m_verbosity_info = new QRadioButton(tr("Info")); m_verbosity_debug = new QRadioButton(tr("Debug")); + m_verbosity_debug->setVisible(Common::Log::MAX_LOGLEVEL == Common::Log::LogLevel::LDEBUG); auto* outputs = new QGroupBox(tr("Logger Outputs")); auto* outputs_layout = new QVBoxLayout; @@ -77,10 +78,7 @@ void LogConfigWidget::CreateWidgets() verbosity_layout->addWidget(m_verbosity_error); verbosity_layout->addWidget(m_verbosity_warning); verbosity_layout->addWidget(m_verbosity_info); - if constexpr (Common::Log::MAX_LOGLEVEL == Common::Log::LogLevel::LDEBUG) - { - verbosity_layout->addWidget(m_verbosity_debug); - } + verbosity_layout->addWidget(m_verbosity_debug); layout->addWidget(outputs); outputs_layout->addWidget(m_out_file);