From a44f8ffd427fa9b23b1f00dbf62d66cd152cd774 Mon Sep 17 00:00:00 2001 From: donovaly Date: Fri, 18 Sep 2020 00:59:33 +0200 Subject: [PATCH] [PD] simplify item handling according to https://github.com/FreeCAD/FreeCAD/commit/2b676020 "QListWidget::takeItem() returns a pointer to the removed item. So a previous call of QListWidget::item() is superfluous" --- src/Mod/PartDesign/Gui/TaskLoftParameters.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/Mod/PartDesign/Gui/TaskLoftParameters.cpp b/src/Mod/PartDesign/Gui/TaskLoftParameters.cpp index 32bd20803f..2106604e87 100644 --- a/src/Mod/PartDesign/Gui/TaskLoftParameters.cpp +++ b/src/Mod/PartDesign/Gui/TaskLoftParameters.cpp @@ -241,10 +241,9 @@ void TaskLoftParameters::onDeleteSection() { // Delete the selected profile int row = ui->listWidgetReferences->currentRow(); - QListWidgetItem* item = ui->listWidgetReferences->item(row); + QListWidgetItem* item = ui->listWidgetReferences->takeItem(row); if (item) { QByteArray data = item->data(Qt::UserRole).toByteArray(); - ui->listWidgetReferences->takeItem(row); delete item; // search inside the list of sections