summaryrefslogtreecommitdiff
authorleseb <leseb>2002-06-16 15:34:57 (UTC)
committer leseb <leseb>2002-06-16 15:34:57 (UTC)
commit1b74249e80cf9f33e52fdb7cc96a114e8be85bc1 (patch) (side-by-side diff)
tree679a1a47c9843e8f89dbe0b3bad532ae60187d72
parent4c1300773b986ad5725492cd26743d542ed16311 (diff)
downloadopie-1b74249e80cf9f33e52fdb7cc96a114e8be85bc1.zip
opie-1b74249e80cf9f33e52fdb7cc96a114e8be85bc1.tar.gz
opie-1b74249e80cf9f33e52fdb7cc96a114e8be85bc1.tar.bz2
Fix memory leaks
Diffstat (more/less context) (ignore whitespace changes)
-rw-r--r--noncore/unsupported/mail2/addresspicker.cpp10
-rw-r--r--noncore/unsupported/mail2/attachdiag.cpp10
-rw-r--r--noncore/unsupported/mail2/configdiag.cpp21
-rw-r--r--noncore/unsupported/mail2/rename.cpp28
4 files changed, 35 insertions, 34 deletions
diff --git a/noncore/unsupported/mail2/addresspicker.cpp b/noncore/unsupported/mail2/addresspicker.cpp
index 178fe27..7f32725 100644
--- a/noncore/unsupported/mail2/addresspicker.cpp
+++ b/noncore/unsupported/mail2/addresspicker.cpp
@@ -101,17 +101,17 @@ void AddressPicker::accept()
}
QString AddressPicker::getNames()
{
QString names = 0;
- AddressPicker *picker = new AddressPicker(0, 0, true);
- picker->showMaximized();
- picker->show();
+ AddressPicker picker(0, 0, true);
+ picker.showMaximized();
+ picker.show();
- int ret = picker->exec();
+ int ret = picker.exec();
if (QDialog::Accepted == ret) {
- return picker->selectedNames;
+ return picker.selectedNames;
}
return 0;
}
diff --git a/noncore/unsupported/mail2/attachdiag.cpp b/noncore/unsupported/mail2/attachdiag.cpp
index bedf3c4..36e1948 100644
--- a/noncore/unsupported/mail2/attachdiag.cpp
+++ b/noncore/unsupported/mail2/attachdiag.cpp
@@ -54,18 +54,18 @@ void AttachDiag::fileSelected(const DocLnk &file)
{
currentFile = file;
}
DocLnk AttachDiag::getFile(QWidget *parent)
{
- AttachDiag *attach = new AttachDiag(parent, 0, true);
- attach->showMaximized();
- attach->show();
+ AttachDiag attach(parent, 0, true);
+ attach.showMaximized();
+ attach.show();
- if (QDialog::Accepted == attach->exec()) {
- return attach->currentFile;
+ if (QDialog::Accepted == attach.exec()) {
+ return attach.currentFile;
}
return DocLnk();
}
diff --git a/noncore/unsupported/mail2/configdiag.cpp b/noncore/unsupported/mail2/configdiag.cpp
index be1f1f4..b89b937 100644
--- a/noncore/unsupported/mail2/configdiag.cpp
+++ b/noncore/unsupported/mail2/configdiag.cpp
@@ -76,18 +76,19 @@ void ConfigDiag::slotFillLists()
for (it = accounts.begin(); it != accounts.end(); it++)
(void) new AccountListItem(accountList, *it);
}
void ConfigDiag::slotNewAccount()
{
- AccountEditor *editor = new AccountEditor(Account(), 0, 0, true);
- editor->showMaximized();
- editor->show();
+ Account account;
+ AccountEditor editor(account, 0, 0, true);
+ editor.showMaximized();
+ editor.show();
- if (QDialog::Accepted == editor->exec()) {
- ConfigFile::updateAccount(editor->_account);
+ if (QDialog::Accepted == editor.exec()) {
+ ConfigFile::updateAccount(editor._account);
slotFillLists();
emit changed();
}
}
void ConfigDiag::slotEditAccount()
@@ -95,18 +96,18 @@ void ConfigDiag::slotEditAccount()
if (!accountList->currentItem()) {
QMessageBox::information(this, tr("Error"), tr("<p>You have to select an account first.</p>"), tr("Ok"));
return;
}
Account account = ((AccountListItem *)accountList->currentItem())->account();
- AccountEditor *editor = new AccountEditor(account, 0, 0, true);
- editor->showMaximized();
- editor->show();
+ AccountEditor editor(account, 0, 0, true);
+ editor.showMaximized();
+ editor.show();
- if (QDialog::Accepted == editor->exec()) {
- ConfigFile::updateAccount(editor->_account);
+ if (QDialog::Accepted == editor.exec()) {
+ ConfigFile::updateAccount(editor._account);
slotFillLists();
emit changed();
}
}
void ConfigDiag::slotDelAccount()
diff --git a/noncore/unsupported/mail2/rename.cpp b/noncore/unsupported/mail2/rename.cpp
index 8c2fde1..627db0b 100644
--- a/noncore/unsupported/mail2/rename.cpp
+++ b/noncore/unsupported/mail2/rename.cpp
@@ -7,33 +7,33 @@ Rename::Rename(QWidget *parent, const char *name, bool modal, WFlags fl)
: RenameBase(parent, name, modal, fl)
{
}
QString Rename::rename(const QString &on, QWidget *parent)
{
- Rename *rename = new Rename(parent, 0, true);
- rename->oldName->setText(on);
- rename->newName->setText(on);
- rename->newName->setFocus();
+ Rename rename(parent, 0, true);
+ rename.oldName->setText(on);
+ rename.newName->setText(on);
+ rename.newName->setFocus();
- if (rename->exec() == QDialog::Accepted) {
- return rename->newName->text();
+ if (rename.exec() == QDialog::Accepted) {
+ return rename.newName->text();
}
return QString(0);
}
QString Rename::getText(const QString &caption, const QString &secondLabel, QWidget *parent)
{
- Rename *rename = new Rename(parent, 0, true);
- rename->firstLabel->hide();
- rename->oldName->hide();
- rename->setCaption(caption);
- rename->secondLabel->setText(secondLabel);
-
- if (rename->exec() == QDialog::Accepted) {
- return rename->newName->text();
+ Rename rename(parent, 0, true);
+ rename.firstLabel->hide();
+ rename.oldName->hide();
+ rename.setCaption(caption);
+ rename.secondLabel->setText(secondLabel);
+
+ if (rename.exec() == QDialog::Accepted) {
+ return rename.newName->text();
}
return QString(0);
}