Fix danmar#455 (Recheck is enabled when no previous results available)

aggro80 · aggro80 · commit 8e0f868a5e32 · 2009-10-13T11:19:00.000+03:00
http://sourceforge.net/apps/trac/cppcheck/ticket/455
diff --git a/gui/mainwindow.cpp b/gui/mainwindow.cpp
@@ -89,6 +89,7 @@ MainWindow::MainWindow() :
 
     mUI.mActionClearResults->setEnabled(false);
     mUI.mActionSave->setEnabled(false);
+    mUI.mActionRecheck->setEnabled(false);
 
 
 
@@ -419,7 +420,10 @@ void MainWindow::EnableCheckButtons(bool enable)
 {
     mUI.mActionStop->setEnabled(!enable);
     mUI.mActionCheckFiles->setEnabled(enable);
-    mUI.mActionRecheck->setEnabled(enable);
+
+    if (!enable || mThread->HasPreviousFiles())
+        mUI.mActionRecheck->setEnabled(enable);
+
     mUI.mActionCheckDirectory->setEnabled(enable);
 }
 
diff --git a/gui/threadhandler.cpp b/gui/threadhandler.cpp
@@ -166,3 +166,10 @@ void ThreadHandler::SaveSettings(QSettings &settings)
     settings.setValue(SETTINGS_CHECK_THREADS, mThreads.size());
 }
 
+bool ThreadHandler::HasPreviousFiles() const
+{
+    if (mLastFiles.size() > 0)
+        return true;
+
+    return false;
+}
diff --git a/gui/threadhandler.h b/gui/threadhandler.h
@@ -96,6 +96,13 @@ class ThreadHandler : public QObject
     */
     bool IsChecking() const;
 
+    /**
+    * @brief Have we checked files already?
+    *
+    * @return true check has been previously run and recheck can be done
+    */
+    bool HasPreviousFiles() const;
+
 signals:
     /**
     * @brief Signal that all threads are done

Original file line number	Diff line number	Diff line change
`@@ -89,6 +89,7 @@ MainWindow::MainWindow() :`
`89`	`89`
`90`	`90`	`mUI.mActionClearResults->setEnabled(false);`
`91`	`91`	`mUI.mActionSave->setEnabled(false);`
	`92`	`+ mUI.mActionRecheck->setEnabled(false);`
`92`	`93`
`93`	`94`
`94`	`95`
`@@ -419,7 +420,10 @@ void MainWindow::EnableCheckButtons(bool enable)`
`419`	`420`	`{`
`420`	`421`	`mUI.mActionStop->setEnabled(!enable);`
`421`	`422`	`mUI.mActionCheckFiles->setEnabled(enable);`
`422`		`- mUI.mActionRecheck->setEnabled(enable);`
	`423`	`+`
	`424`	`+ if (!enable \|\| mThread->HasPreviousFiles())`
	`425`	`+ mUI.mActionRecheck->setEnabled(enable);`
	`426`	`+`
`423`	`427`	`mUI.mActionCheckDirectory->setEnabled(enable);`
`424`	`428`	`}`
`425`	`429`