From d7006ccefa50984698a10aa28728420e356f61fe Mon Sep 17 00:00:00 2001 From: LoRd_MuldeR Date: Mon, 11 Dec 2017 20:00:04 +0100 Subject: [PATCH] Removed unreachable code. --- src/Dialog_MainWindow.cpp | 53 +++++++++++++++-------------------------------- 1 file changed, 17 insertions(+), 36 deletions(-) diff --git a/src/Dialog_MainWindow.cpp b/src/Dialog_MainWindow.cpp index 1e8e3ee7..67a8c1d9 100644 --- a/src/Dialog_MainWindow.cpp +++ b/src/Dialog_MainWindow.cpp @@ -1583,41 +1583,22 @@ void MainWindow::windowShown(void) } //Check for AAC support - const int aacEncoder = EncoderRegistry::getAacEncoder(); - if(aacEncoder == SettingsModel::AAC_ENCODER_NERO) - { - if(m_settings->neroAacNotificationsEnabled()) - { - if(lamexp_tools_version("neroAacEnc.exe") < lamexp_toolver_neroaac()) - { - QString messageText; - messageText += NOBR(tr("LameXP detected that your version of the Nero AAC encoder is outdated!")).append("
"); - messageText += NOBR(tr("The current version available is %1 (or later), but you still have version %2 installed.").arg(lamexp_version2string("?.?.?.?", lamexp_toolver_neroaac(), tr("n/a")), lamexp_version2string("?.?.?.?", lamexp_tools_version("neroAacEnc.exe"), tr("n/a")))).append("

"); - messageText += NOBR(tr("You can download the latest version of the Nero AAC encoder from the Nero website at:")).append("
"); - messageText += QString("").append(LINK(AboutDialog::neroAacUrl)).append("

"); - messageText += NOBR(tr("(Hint: Please ignore the name of the downloaded ZIP file and check the included 'changelog.txt' instead!)")).append("
"); - QMessageBox::information(this, tr("AAC Encoder Outdated"), messageText); - } - } - } - else - { - if(m_settings->neroAacNotificationsEnabled() && (aacEncoder <= SettingsModel::AAC_ENCODER_NONE)) + if(m_settings->neroAacNotificationsEnabled() && (EncoderRegistry::getAacEncoder() <= SettingsModel::AAC_ENCODER_NONE)) + { + QString appPath = QDir(QCoreApplication::applicationDirPath()).canonicalPath(); + if(appPath.isEmpty()) appPath = QCoreApplication::applicationDirPath(); + QString messageText; + messageText += NOBR(tr("The Nero AAC encoder could not be found. AAC encoding support will be disabled.")).append("
"); + messageText += NOBR(tr("Please put 'neroAacEnc.exe', 'neroAacDec.exe' and 'neroAacTag.exe' into the LameXP directory!")).append("

"); + messageText += QString("").append(NOBR(tr("Your LameXP install directory is located here:"))).append("
"); + messageText += QString("%1

").arg(FSLINK(QDir::toNativeSeparators(appPath))); + messageText += QString("").append(NOBR(tr("You can download the Nero AAC encoder for free from this website:"))).append("
"); + messageText += QString("").append(LINK(AboutDialog::neroAacUrl)).append("

"); + messageText += QString("").append(NOBR(tr("Note: Nero AAC encoder version %1 or newer is required to enable AAC encoding support!").arg(lamexp_version2string("v?.?.?.?", lamexp_toolver_neroaac(), "n/a")))).append("
"); + if(QMessageBox::information(this, tr("AAC Support Disabled"), messageText, tr("Discard"), tr("Don't Show Again")) == 1) { - QString appPath = QDir(QCoreApplication::applicationDirPath()).canonicalPath(); - if(appPath.isEmpty()) appPath = QCoreApplication::applicationDirPath(); - QString messageText; - messageText += NOBR(tr("The Nero AAC encoder could not be found. AAC encoding support will be disabled.")).append("
"); - messageText += NOBR(tr("Please put 'neroAacEnc.exe', 'neroAacDec.exe' and 'neroAacTag.exe' into the LameXP directory!")).append("

"); - messageText += QString("").append(NOBR(tr("Your LameXP install directory is located here:"))).append("
"); - messageText += QString("%1

").arg(FSLINK(QDir::toNativeSeparators(appPath))); - messageText += QString("").append(NOBR(tr("You can download the Nero AAC encoder for free from this website:"))).append("
"); - messageText += QString("").append(LINK(AboutDialog::neroAacUrl)).append("
"); - if(QMessageBox::information(this, tr("AAC Support Disabled"), messageText, tr("Discard"), tr("Don't Show Again")) == 1) - { - m_settings->neroAacNotificationsEnabled(false); - ui->actionDisableNeroAacNotifications->setChecked(!m_settings->neroAacNotificationsEnabled()); - } + m_settings->neroAacNotificationsEnabled(false); + ui->actionDisableNeroAacNotifications->setChecked(!m_settings->neroAacNotificationsEnabled()); } } @@ -2277,8 +2258,8 @@ void MainWindow::checkForBetaUpdatesActionTriggered(bool checked) } else { - QMessageBox::information(this, tr("Beta Updates"), NOBR(tr("LameXP will not check for Beta (pre-release) updates from now on."))); - m_settings->autoUpdateCheckBeta(false); + QMessageBox::information(this, tr("Beta Updates"), NOBR(tr("LameXP will not check for Beta (pre-release) updates from now on."))); + m_settings->autoUpdateCheckBeta(false); } ui->actionCheckForBetaUpdates->setChecked(m_settings->autoUpdateCheckBeta()); -- 2.11.4.GIT