mirror of
https://github.com/subsurface/subsurface.git
synced 2024-11-28 21:20:19 +00:00
b22f1da59e
Subsurface creates a lot of singleton instances on demand, but nothing ever deleted them. Since they are singletons, these memory allocations are technically not leaks. However, they clutter the output in valgrind and other memory analysers, hiding the real issues. The solution is to delete these items at exit. For the models and for gettextFromC, the solution is to use a QScopedPointer, which will delete its payload when it gets destroyed. For the dialogs and other widgets, we can't do that: they need to be deleted before QApplication exits, so we just set the parent in all of them to the main window. Signed-off-by: Thiago Macieira <thiago@macieira.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
67 lines
1.6 KiB
C++
67 lines
1.6 KiB
C++
#include "divecomputermanagementdialog.h"
|
|
#include "models.h"
|
|
#include "mainwindow.h"
|
|
#include <QMessageBox>
|
|
#include "../qthelper.h"
|
|
#include "../helpers.h"
|
|
|
|
DiveComputerManagementDialog::DiveComputerManagementDialog(QWidget* parent, Qt::WindowFlags f): QDialog(parent, f),
|
|
model(0)
|
|
{
|
|
ui.setupUi(this);
|
|
init();
|
|
connect(ui.tableView, SIGNAL(clicked(QModelIndex)), this, SLOT(tryRemove(QModelIndex)));
|
|
}
|
|
|
|
void DiveComputerManagementDialog::init()
|
|
{
|
|
if (model)
|
|
delete model;
|
|
model = new DiveComputerModel(dcList.dcMap);
|
|
ui.tableView->setModel(model);
|
|
}
|
|
|
|
DiveComputerManagementDialog* DiveComputerManagementDialog::instance()
|
|
{
|
|
static DiveComputerManagementDialog *self = new DiveComputerManagementDialog(mainWindow());
|
|
self->setAttribute(Qt::WA_QuitOnClose, false);
|
|
return self;
|
|
}
|
|
|
|
void DiveComputerManagementDialog::update()
|
|
{
|
|
model->update();
|
|
ui.tableView->resizeColumnsToContents();
|
|
ui.tableView->setColumnWidth(DiveComputerModel::REMOVE, 22);
|
|
layout()->activate();
|
|
}
|
|
|
|
void DiveComputerManagementDialog::tryRemove(const QModelIndex& index)
|
|
{
|
|
if (index.column() != DiveComputerModel::REMOVE)
|
|
return;
|
|
|
|
QMessageBox::StandardButton response = QMessageBox::question(
|
|
this, TITLE_OR_TEXT(
|
|
tr("Remove the selected Dive Computer?"),
|
|
tr("Are you sure that you want to \n remove the selected dive computer?")),
|
|
QMessageBox::Ok | QMessageBox::Cancel
|
|
);
|
|
|
|
if (response == QMessageBox::Ok)
|
|
model->remove(index);
|
|
}
|
|
|
|
void DiveComputerManagementDialog::accept()
|
|
{
|
|
model->keepWorkingList();
|
|
hide();
|
|
close();
|
|
}
|
|
|
|
void DiveComputerManagementDialog::reject()
|
|
{
|
|
model->dropWorkingList();
|
|
hide();
|
|
close();
|
|
}
|