From eede69e313d49b7c5450c20e918ae51ac1b2cad0 Mon Sep 17 00:00:00 2001 From: Jarek Kobus Date: Tue, 17 Nov 2020 00:08:13 +0100 Subject: [PATCH] Don't create a temporary list of values for qDeleteAll It's enough to pass QHash as an input for qDeleteAll, it will delete all values and won't touch keys. Change-Id: I5a924f4d59787ac3fd1ffaf07709ab91ba96c7c6 Reviewed-by: Tim Jenssen --- src/plugins/marketplace/productlistmodel.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/plugins/marketplace/productlistmodel.cpp b/src/plugins/marketplace/productlistmodel.cpp index 476b7f1a92a..115a86dfda6 100644 --- a/src/plugins/marketplace/productlistmodel.cpp +++ b/src/plugins/marketplace/productlistmodel.cpp @@ -193,7 +193,7 @@ SectionedProducts::SectionedProducts(QWidget *parent) SectionedProducts::~SectionedProducts() { - qDeleteAll(m_gridViews.values()); + qDeleteAll(m_gridViews); delete m_productDelegate; delete m_gridModel; }