Don't leak memory in qgsnodeeditor delegates

qgis · Dec 5, 2017 · efad2f6 · efad2f6
1 parent 2b3fbc7
commit efad2f6
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 5 deletions.
diff --git a/src/app/nodetool/qgsnodeeditor.cpp b/src/app/nodetool/qgsnodeeditor.cpp
@@ -297,11 +297,11 @@ QgsNodeEditor::QgsNodeEditor(
 
   mTableView->setSelectionMode( QTableWidget::ExtendedSelection );
   mTableView->setSelectionBehavior( QTableWidget::SelectRows );
-  mTableView->setItemDelegateForColumn( 0, new CoordinateItemDelegate() );
-  mTableView->setItemDelegateForColumn( 1, new CoordinateItemDelegate() );
-  mTableView->setItemDelegateForColumn( 2, new CoordinateItemDelegate() );
-  mTableView->setItemDelegateForColumn( 3, new CoordinateItemDelegate() );
-  mTableView->setItemDelegateForColumn( 4, new CoordinateItemDelegate() );
+  mTableView->setItemDelegateForColumn( 0, new CoordinateItemDelegate( this ) );
+  mTableView->setItemDelegateForColumn( 1, new CoordinateItemDelegate( this ) );
+  mTableView->setItemDelegateForColumn( 2, new CoordinateItemDelegate( this ) );
+  mTableView->setItemDelegateForColumn( 3, new CoordinateItemDelegate( this ) );
+  mTableView->setItemDelegateForColumn( 4, new CoordinateItemDelegate( this ) );
 
   setWidget( mTableView );
 
@@ -396,6 +396,12 @@ void QgsNodeEditor::keyPressEvent( QKeyEvent *e )
 // CoordinateItemDelegate
 //
 
+CoordinateItemDelegate::CoordinateItemDelegate( QObject *parent )
+  : QStyledItemDelegate( parent )
+{
+
+}
+
 QString CoordinateItemDelegate::displayText( const QVariant &value, const QLocale &locale ) const
 {
   return locale.toString( value.toDouble(), 'f', 4 );

diff --git a/src/app/nodetool/qgsnodeeditor.h b/src/app/nodetool/qgsnodeeditor.h
@@ -107,6 +107,9 @@ class CoordinateItemDelegate : public QStyledItemDelegate
     Q_OBJECT
 
   public:
+
+    explicit CoordinateItemDelegate( QObject *parent = nullptr );
+
     QString displayText( const QVariant &value, const QLocale &locale ) const override;
 
   protected: