summaryrefslogtreecommitdiffstats
path: root/examples/sql/sqlbrowser/browser.cpp
diff options
context:
space:
mode:
authorMarc Mutz <[email protected]>2024-12-31 10:03:41 +0100
committerMarc Mutz <[email protected]>2025-01-02 07:50:57 +0100
commit0da2c2c4ef2219967db87021eece2a60b6e207af (patch)
treefc68cf81cb463b075f1855efd59bfed8b4789c9a /examples/sql/sqlbrowser/browser.cpp
parent9c099ef942216d01261c26b60e3727a2a467f12a (diff)
sqlbrowser example: use unique_ptr to hold m_ui
The old code used manual memory mangement (raw new/delete) to (de)allocate the Ui struct. This is so 80s. Use an owning smart pointer to manage the memory. Ordinarily, this would have been QScopedPointer, but seeing as that doesn't have a create() method to hide the raw new, use std::unique_ptr and std::make_unique() instead. Amends 2690822428deec4f0c08f4d118d69a7c6036369e. Pick-to: 6.9 6.8 Change-Id: Icabb9154eb38630855e14094b958af0214516f6b Reviewed-by: Volker Hilsheimer <[email protected]>
Diffstat (limited to 'examples/sql/sqlbrowser/browser.cpp')
-rw-r--r--examples/sql/sqlbrowser/browser.cpp6
1 files changed, 2 insertions, 4 deletions
diff --git a/examples/sql/sqlbrowser/browser.cpp b/examples/sql/sqlbrowser/browser.cpp
index 9a99999702b..3aad9fb890b 100644
--- a/examples/sql/sqlbrowser/browser.cpp
+++ b/examples/sql/sqlbrowser/browser.cpp
@@ -18,7 +18,7 @@
Browser::Browser(QWidget *parent)
: QWidget(parent)
- , m_ui(new Ui::Browser)
+ , m_ui{std::make_unique<Ui::Browser>()}
{
m_ui->setupUi(this);
@@ -60,9 +60,7 @@ Browser::Browser(QWidget *parent)
}
Browser::~Browser()
-{
- delete m_ui;
-}
+ = default;
void Browser::exec()
{