From b6a9d7973d823b2a8e513a291d7863fdd3df33ff Mon Sep 17 00:00:00 2001 From: wmayer Date: Sat, 9 Nov 2013 13:51:21 +0100 Subject: [PATCH] + Fix memory leak in units calculator --- src/Gui/DlgUnitsCalculatorImp.cpp | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/Gui/DlgUnitsCalculatorImp.cpp b/src/Gui/DlgUnitsCalculatorImp.cpp index 885268e774..73c03798ff 100644 --- a/src/Gui/DlgUnitsCalculatorImp.cpp +++ b/src/Gui/DlgUnitsCalculatorImp.cpp @@ -88,15 +88,15 @@ void DlgUnitsCalculator::valueChanged(const Base::Quantity& quant) QString out(QString::fromAscii("%1 %2")); out = out.arg(value).arg(this->UnitInput->text()); this->ValueOutput->setText(out); - QPalette *palette = new QPalette(); - palette->setColor(QPalette::Base,QColor(200,255,200)); - this->ValueOutput->setPalette(*palette); + QPalette palette; + palette.setColor(QPalette::Base,QColor(200,255,200)); + this->ValueOutput->setPalette(palette); }else{ //this->ValueOutput->setValue(quant); this->ValueOutput->setText(QString::fromAscii(quant.getUserString().c_str())); - QPalette *palette = new QPalette(); - palette->setColor(QPalette::Base,QColor(200,255,200)); - this->ValueOutput->setPalette(*palette); + QPalette palette; + palette.setColor(QPalette::Base,QColor(200,255,200)); + this->ValueOutput->setPalette(palette); } actValue = quant; @@ -104,9 +104,9 @@ void DlgUnitsCalculator::valueChanged(const Base::Quantity& quant) void DlgUnitsCalculator::parseError(const QString& errorText) { - QPalette *palette = new QPalette(); - palette->setColor(QPalette::Base,QColor(255,200,200)); - this->ValueOutput->setPalette(*palette); + QPalette palette; + palette.setColor(QPalette::Base,QColor(255,200,200)); + this->ValueOutput->setPalette(palette); this->ValueOutput->setText(QString()); }