From 857da2996795fe39d5d60fe3dee6244bd8ca76d7 Mon Sep 17 00:00:00 2001 From: Sebastian Date: Tue, 6 Mar 2018 22:49:33 +0100 Subject: [PATCH] GUI: Remove redundant code (#1102) The checked state of the addon checkboxes is already set in the updateAddonCheckBox function directly above the removed code. I do not see any reason to set it again and only for three of the four checkboxes. --- gui/projectfiledialog.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/gui/projectfiledialog.cpp b/gui/projectfiledialog.cpp index bfc9ad9fb..b990f8af6 100644 --- a/gui/projectfiledialog.cpp +++ b/gui/projectfiledialog.cpp @@ -165,9 +165,6 @@ void ProjectFileDialog::loadFromProjectFile(const ProjectFile *projectFile) updateAddonCheckBox(mUI.mAddonCert, projectFile, dataDir, "cert"); updateAddonCheckBox(mUI.mAddonMisra, projectFile, dataDir, "misra"); - mUI.mAddonY2038->setChecked(projectFile->getAddons().contains("y2038")); - mUI.mAddonCert->setChecked(projectFile->getAddons().contains("cert")); - mUI.mAddonMisra->setChecked(projectFile->getAddons().contains("misra")); mUI.mToolClangAnalyzer->setChecked(projectFile->getClangAnalyzer()); mUI.mToolClangTidy->setChecked(projectFile->getClangTidy()); if (CheckThread::clangTidyCmd().isEmpty()) {