Re: [PATCH] comctl32/progress: Fix wrapping of values in PBM_STEPIT and add tests

[Fabian Maurer]

On Montag, 26. Februar 2018 23:21:07 CET Nikolay Sivov wrote:
> On 2/27/2018 12:14 AM, Fabian Maurer wrote:
> > -
> > - SendMessageA(progress, PBM_SETSTEP, TEST_STEP_SMALL, 0);
> > - SendMessageA(progress, PBM_STEPIT, 0, 0);
>
> Maybe have this as SETPOS instead?
> > - for (i = 1; i < 15; i++)
> > - {
> > -
> >    pos_actual = SendMessageA(progress, PBM_STEPIT, 0, 0);
>
> That's previous position, not actual/current.

I thought it's easier to work with that way, I've changed it in the rewrite of the patch.

Regards,
Fabian Maurer