Correct progress display on searchdeterministiczerocoin

Zannick · Zannick · commit cdc23495d27e · 2022-04-07T22:07:50.000-07:00
The progress items of the individual threads have already accounted for
their start points, so we don't need to subtract the overall start point
again.
diff --git a/src/veil/zerocoin/zwallet.cpp b/src/veil/zerocoin/zwallet.cpp
@@ -392,7 +392,9 @@ bool CzWallet::DeterministicSearch(int nCountStart, int nCountEnd, int* nThreade
                 }
             }
 
-            int percentageDone = std::max(1, std::min(99, (int)((progress_current - progress_begin) / (progress_end - progress_begin) * 100)));
+            // items complete = progress_current
+            // items total = progress_end - progress_begin
+            int percentageDone = std::max(1, std::min(99, (int)(progress_current / (progress_end - progress_begin) * 100)));
             uiInterface.ShowProgress(_("Searching..."), percentageDone, 0);
         }
         if (nThreadedProgress == nullptr) {

Original file line number	Diff line number	Diff line change
`@@ -392,7 +392,9 @@ bool CzWallet::DeterministicSearch(int nCountStart, int nCountEnd, int* nThreade`
`392`	`392`	`}`
`393`	`393`	`}`
`394`	`394`
`395`		`- int percentageDone = std::max(1, std::min(99, (int)((progress_current - progress_begin) / (progress_end - progress_begin) * 100)));`
	`395`	`+ // items complete = progress_current`
	`396`	`+ // items total = progress_end - progress_begin`
	`397`	`+ int percentageDone = std::max(1, std::min(99, (int)(progress_current / (progress_end - progress_begin) * 100)));`
`396`	`398`	`uiInterface.ShowProgress(_("Searching..."), percentageDone, 0);`
`397`	`399`	`}`
`398`	`400`	`if (nThreadedProgress == nullptr) {`