Vitaly Lipatov <lav(a)etersoft.ru> 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. -- Alexandre Julliard julliard(a)winehq.org