[utilities/kate] addons/katebuild-plugin: KateBuildConfigPage: do not leak UI instance

Kåre Särs <[email protected]> Wed, 5 Aug 2026 06:00:35 +0000 (UTC)
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit 0b871bc25895751bceaef467aedf6fd66c34e657 by Kåre Särs, on behalf of Friedrich W. H. Kossebau.
Committed on 05/08/2026 at 05:43.
Pushed by sars into branch 'master'.

KateBuildConfigPage: do not leak UI instance

M  +5    -0    addons/katebuild-plugin/buildconfig.cpp
M  +1    -3    addons/katebuild-plugin/buildconfig.h

https://invent.kde.org/utilities/kate/-/commit/0b871bc25895751bceaef467aedf6fd66c34e657

diff --git a/addons/katebuild-plugin/buildconfig.cpp b/addons/katebuild-plugin/buildconfig.cpp
index ad647a5c12..19d2b37dae 100644
--- a/addons/katebuild-plugin/buildconfig.cpp
+++ b/addons/katebuild-plugin/buildconfig.cpp
@@ -42,6 +42,11 @@ KateBuildConfigPage::KateBuildConfigPage(KateBuildPlugin *plugin, QWidget *paren
     connect(ui->allowedAndBlockedCommands, &QWidget::customContextMenuRequested, this, &KateBuildConfigPage::showContextMenuAllowedBlocked);
 }
 
+KateBuildConfigPage::~KateBuildConfigPage()
+{
+    delete ui;
+}
+
 /******************************************************************/
 QString KateBuildConfigPage::name() const
 {
diff --git a/addons/katebuild-plugin/buildconfig.h b/addons/katebuild-plugin/buildconfig.h
index d4a601ecb7..c9093b2a87 100644
--- a/addons/katebuild-plugin/buildconfig.h
+++ b/addons/katebuild-plugin/buildconfig.h
@@ -25,9 +25,7 @@ class KateBuildConfigPage : public KTextEditor::ConfigPage
 {
 public:
     explicit KateBuildConfigPage(KateBuildPlugin *plugin, QWidget *parent = nullptr);
-    ~KateBuildConfigPage() override
-    {
-    }
+    ~KateBuildConfigPage() override;
 
     QString name() const override;
     QString fullName() const override;