commit df8780356bdde662dc743ea51753fe8bb13abeab
author Dmitry Timoshkov <dmitry@codeweavers.com> Thu Oct 29 20:46:34 2009 +0800
committer Alexandre Julliard <julliard@winehq.org> Thu Oct 29 15:48:27 2009 +0100
tree 66a3a64e4d8188bf797afade70ecc053b01140fa
parent 37f3f3d92a8854104de0ef6774bc54f4a1400354

msi: Limit the progress to its maximal value, use MulDiv for better results.

dlls/msi/dialog.c

diff --git a/dlls/msi/dialog.c b/dlls/msi/dialog.c
index 3f1b571..88cdfa9 100644
--- a/dlls/msi/dialog.c
+++ b/dlls/msi/dialog.c
@@ -606,6 +606,8 @@
         func = MSI_RecordGetInteger( rec , 1 );
         val = MSI_RecordGetInteger( rec , 2 );
 
+        TRACE("progress: func %u, val %u\n", func, val);
+
         switch (func)
         {
         case 0: /* init */
@@ -618,10 +620,12 @@
             break;
         case 2: /* move */
             ctrl->progress_current += val;
-            SendMessageW(ctrl->hwnd, PBM_SETPOS, 100*(ctrl->progress_current/ctrl->progress_max), 0);
+            if (ctrl->progress_current > ctrl->progress_max)
+                ctrl->progress_current = ctrl->progress_max;
+            SendMessageW(ctrl->hwnd, PBM_SETPOS, MulDiv(100, ctrl->progress_current, ctrl->progress_max), 0);
             break;
         default:
-            ERR("Unknown progress message %d\n", func);
+            FIXME("Unknown progress message %u\n", func);
             break;
         }
     }