Re: USER32: result of ScrollBar functions review (4th try)

[Alexandre Julliard]

Vitaly Lipatov  writes:
> @@ -1728,12 +1821,15 @@
>      SCROLLINFO info;
>  TRACE("hwnd=%p nBar=%d min=%d max=%d, bRedraw=%d\n", hwnd, nBar, minVal, maxVal, bRedraw);
> -
> - info.cbSize = sizeof(info);
>    info.fMask  = SIF_RANGE;
>    info.nMin   = minVal;
>    info.nMax   = maxVal;
>    SetScrollInfo( hwnd, nBar, &info, bRedraw );
> - if (nBar == SB_CTL)
> - FIXME("needs to send SMB_SETRANGE when SB_CTL\n");
> -
>    //return SendMessageW(hwnd, SBM_SETRANGE, (WPARAM)0, (LPARAM)0);
> return TRUE;

There's no reason to add a FIXME, calling SetScrollInfo is perfectly
fine and it's what Windows does too, we even have a test for it.

Also please send separate changes as separate patches, especially when
you are renaming functions which causes a lot of changes all over and
makes it hard to see what else is changing.