Cleanup: fix leak in delegates

A copy of a C-string was assigned to a QString. The copy was never
freed. Instead, assign the C-string directly. This does the right
thing.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
This commit is contained in:
Berthold Stoeger 2019-04-27 17:27:37 +02:00 committed by Dirk Hohndel
parent cab0147093
commit 7b83eeadff

View file

@ -294,7 +294,7 @@ QWidget *TankInfoDelegate::createEditor(QWidget *parent, const QStyleOptionViewI
QWidget *delegate = ComboBoxDelegate::createEditor(parent, option, index); QWidget *delegate = ComboBoxDelegate::createEditor(parent, option, index);
CylindersModel *mymodel = qobject_cast<CylindersModel *>(currCombo.model); CylindersModel *mymodel = qobject_cast<CylindersModel *>(currCombo.model);
cylinder_t *cyl = mymodel->cylinderAt(index); cylinder_t *cyl = mymodel->cylinderAt(index);
currCylinderData.type = copy_string(cyl->type.description); currCylinderData.type = cyl->type.description;
currCylinderData.pressure = cyl->type.workingpressure.mbar; currCylinderData.pressure = cyl->type.workingpressure.mbar;
currCylinderData.size = cyl->type.size.mliter; currCylinderData.size = cyl->type.size.mliter;
MainWindow::instance()->graphics->setReplot(false); MainWindow::instance()->graphics->setReplot(false);
@ -372,7 +372,7 @@ QWidget *WSInfoDelegate::createEditor(QWidget *parent, const QStyleOptionViewIte
QWidget *editor = ComboBoxDelegate::createEditor(parent, option, index); QWidget *editor = ComboBoxDelegate::createEditor(parent, option, index);
WeightModel *mymodel = qobject_cast<WeightModel *>(currCombo.model); WeightModel *mymodel = qobject_cast<WeightModel *>(currCombo.model);
weightsystem_t *ws = mymodel->weightSystemAt(index); weightsystem_t *ws = mymodel->weightSystemAt(index);
currWeight.type = copy_string(ws->description); currWeight.type = ws->description;
currWeight.weight = ws->weight.grams; currWeight.weight = ws->weight.grams;
return editor; return editor;
} }