Fixed danmar#1831 (GUI: no progress bar when rechecking files)

kimmov · kimmov · commit f60a4699324b · 2010-07-07T12:25:47.000+03:00
Need to initialize the checking also when rechecking.
diff --git a/gui/mainwindow.cpp b/gui/mainwindow.cpp
@@ -416,6 +416,11 @@ void MainWindow::ReCheck()
 {
     ClearResults();
     EnableCheckButtons(false);
+
+    const int filesCount = mThread->GetPreviousFilesCount();
+    Q_ASSERT(filesCount > 0); // If no files should not be able to recheck
+    mUI.mResults->CheckingStarted(filesCount);
+
     mThread->Check(GetCppcheckSettings(), true);
 }
 
diff --git a/gui/threadhandler.cpp b/gui/threadhandler.cpp
@@ -173,3 +173,8 @@ bool ThreadHandler::HasPreviousFiles() const
 
     return false;
 }
+
+int ThreadHandler::GetPreviousFilesCount() const
+{
+    return mLastFiles.size();
+}
diff --git a/gui/threadhandler.h b/gui/threadhandler.h
@@ -103,6 +103,13 @@ class ThreadHandler : public QObject
     */
     bool HasPreviousFiles() const;
 
+    /**
+    * @brief Return count of files we checked last time.
+    *
+    * @return count of files that were checked last time.
+    */
+    int GetPreviousFilesCount() const;
+
 signals:
     /**
     * @brief Signal that all threads are done

Original file line number	Diff line number	Diff line change
`@@ -416,6 +416,11 @@ void MainWindow::ReCheck()`
`416`	`416`	`{`
`417`	`417`	`ClearResults();`
`418`	`418`	`EnableCheckButtons(false);`
	`419`	`+`
	`420`	`+ const int filesCount = mThread->GetPreviousFilesCount();`
	`421`	`+ Q_ASSERT(filesCount > 0); // If no files should not be able to recheck`
	`422`	`+ mUI.mResults->CheckingStarted(filesCount);`
	`423`	`+`
`419`	`424`	`mThread->Check(GetCppcheckSettings(), true);`
`420`	`425`	`}`
`421`	`426`
Original file line number	Diff line number	Diff line change
`@@ -173,3 +173,8 @@ bool ThreadHandler::HasPreviousFiles() const`
`173`	`173`
`174`	`174`	`return false;`
`175`	`175`	`}`
	`176`	`+`
	`177`	`+int ThreadHandler::GetPreviousFilesCount() const`
	`178`	`+{`
	`179`	`+ return mLastFiles.size();`
	`180`	`+}`