Set the maximum first when updating the progresslogger progress bar as otherwise the new value will get dropped if outside of the valid range. Fixes the progress showing 0% after downloading the manual.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@21302 a1c6a512-1295-4272-9138-f99709370657

1 files changed, 4 insertions, 1 deletions

diff --git a/rbutil/rbutilqt/progressloggergui.cpp b/rbutil/rbutilqt/progressloggergui.cpp
index 0358d75..a6405a9 100644
--- a/rbutil/rbutilqt/progressloggergui.cpp
+++ b/rbutil/rbutilqt/progressloggergui.cpp
@@ -61,8 +61,11 @@ void ProgressLoggerGui::addItem(const QString &text, int flag)
 
 void ProgressLoggerGui::setProgress(int value, int max)
 {
-    setProgressValue(value);
+    // set maximum first to avoid setting a value outside of the max range.
+    // If the current value is outside of the valid range QProgressBar
+    // calls reset() internally.
     setProgressMax(max);
+    setProgressValue(value);
 }