walkero
/
cppcheck
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

Refactoring: early return

This commit is contained in:
Daniel Marjamäki 2018-08-08 09:00:04 +02:00
parent 80290a15e0
commit 90a3759479
1 changed files with 19 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

39
gui/mainwindow.cpp
View File

@ -1650,28 +1650,27 @@ void MainWindow::enableProjectOpenActions(bool enable)
void MainWindow::openRecentProject() void MainWindow::openRecentProject()
{ {
QAction *action = qobject_cast<QAction *>(sender()); QAction *action = qobject_cast<QAction *>(sender());
if (action) { if (!action)
const QString project = action->data().toString(); return;
QFileInfo inf(project); const QString project = action->data().toString();
if (inf.exists()) { QFileInfo inf(project);
loadProjectFile(project); if (inf.exists()) {
} else { loadProjectFile(project);
const QString text(tr("The project file\n\n%1\n\n could not be found!\n\n" } else {
"Do you want to remove the file from the recently " const QString text(tr("The project file\n\n%1\n\n could not be found!\n\n"
"used projects -list?").arg(project)); "Do you want to remove the file from the recently "
"used projects -list?").arg(project));
QMessageBox msg(QMessageBox::Warning, QMessageBox msg(QMessageBox::Warning,
tr("Cppcheck"), tr("Cppcheck"),
text, text,
QMessageBox::Yes | QMessageBox::No, QMessageBox::Yes | QMessageBox::No,
this); this);
msg.setDefaultButton(QMessageBox::No);
int rv = msg.exec();
if (rv == QMessageBox::Yes) {
removeProjectMRU(project);
}
msg.setDefaultButton(QMessageBox::No);
int rv = msg.exec();
if (rv == QMessageBox::Yes) {
removeProjectMRU(project);
} }
} }
} }